Skip to content

ALFREDAPI-585: Add create with id (for CG rollback mechanism) - #267

Merged
WimCrols merged 12 commits into
master-5.xfrom
ALFREDAPI-585_ethiasrollback-createwithid
Jun 23, 2026
Merged

ALFREDAPI-585: Add create with id (for CG rollback mechanism)#267
WimCrols merged 12 commits into
master-5.xfrom
ALFREDAPI-585_ethiasrollback-createwithid

Conversation

@WimCrols

@WimCrols WimCrols commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Note

Note the base branch: we merge into master-5.x

Fixes https://xenitsupport.jira.com/browse/ALFREDAPI-585

  • Is CHANGELOG.md extended?
  • Does this PR avoid breaking the API?
    Breaking changes include adding, changing or removing endpoints and/or JSON objects used in requests and responses.
  • Are all Alfresco services wired via ServiceRegistry (and not per service), and is ServiceRegistry autowired with @Qualifier("ServiceRegistry")?
  • Does the PR comply to REST HTTP result codes policy outlined in the user guide?
  • Is error handling done through a method annotated with @ExceptionHandler in the webscript classes?
  • Does the PR follow our coding styleguide and other active procedures?
  • Is usage of this. prefix avoided?

See README.md for full explanation.

@WimCrols WimCrols changed the title ALFREDAPI-585: Add failing test for create with id ALFREDAPI-585: Add create with id (for CG rollback mechanism) Jun 8, 2026
@WimCrols
WimCrols force-pushed the ALFREDAPI-585_ethiasrollback-createwithid branch from 1ba65e7 to 96dc593 Compare June 10, 2026 13:56
@WimCrols
WimCrols force-pushed the ALFREDAPI-585_ethiasrollback-createwithid branch from 22e0a11 to 1d851f4 Compare June 15, 2026 14:25
@WimCrols
WimCrols marked this pull request as ready for review June 16, 2026 06:56
@WimCrols
WimCrols force-pushed the ALFREDAPI-585_ethiasrollback-createwithid branch from 1d851f4 to 2454504 Compare June 16, 2026 07:12
Comment thread .github/workflows/ci.yml
Comment thread CHANGELOG.md Outdated

@Brobrechts Brobrechts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general this looks good, I presume most is cherry picked from future and fixes the harbor things (all fine)

The java 21 JDK, should be fine if the specific versions respect the toolchain.

This one also only releases > 7.4 versions looking at build.gradle

Comment thread build.gradle Outdated
Comment thread build.gradle

@Brobrechts Brobrechts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed with Wim crols.

This is also isolated to 7x to keep it buildable.

@WimCrols
WimCrols merged commit 5914ce3 into master-5.x Jun 23, 2026
12 checks passed
@WimCrols
WimCrols deleted the ALFREDAPI-585_ethiasrollback-createwithid branch June 23, 2026 11:13
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants