Skip to content

fix: The administration declaration's comment describes the overwrite reload EXO-91183 - #233

Merged
boubaker merged 1 commit into
developfrom
backport/EXO-91183-portal-configuration
Oct 27, 2026
Merged

boubaker merged 1 commit into
developfrom
backport/EXO-91183-portal-configuration

Conversation

@boubaker

Copy link
Copy Markdown
Member

Symptom: the comment on the administration declaration of portal-configuration.xml says that reloading only the exo part of the site makes the PageImporter remove Meeds' administration pages, and recommends io.meeds.administration.portalConfig.metadata.importmode=overwrite to reload either part. With Meeds-io/portal#1338 the first is false, and the recommended property also reloads the global site, which the comment doesn't say.

Cause: with Meeds-io/portal#1338, a startup that runs an overwrite declaration of a site imports the pages and the navigation of every declaration of that site, and removes the pages and nodes none of them defines. The io.meeds.administration properties also set the mode of this file's global declaration.

Fix: the comment says that, with overwrite, either property reloads the pages and the navigation of all the administration declarations, Meeds' included, and removes the pages and nodes none of them defines, and that the io.meeds.administration property reloads global as well. Comment only; xmllint --noout on the file at 943e39f: no output (well-formed).

Release order: merge with or after Meeds-io/portal#1338, whose behaviour the comment describes.

Knowledge: Meeds-io/eng-standards#409, #560, #586, #587

This change is classified N3: no REST/DAO/schema/ACL surface touched (a comment in a configuration file).

🤖 Generated with Claude Code

… reload EXO-91183 (#232)

**Symptom**: the comment on the `administration` declaration of
`portal-configuration.xml` says that reloading only the exo part of the
site makes the `PageImporter` remove Meeds' administration pages, and
recommends
`io.meeds.administration.portalConfig.metadata.importmode=overwrite` to
reload either part. With Meeds-io/portal#1338 the first is false, and
the recommended property also reloads the `global` site, which the
comment doesn't say.

**Cause**: with Meeds-io/portal#1338, a startup that runs an `overwrite`
declaration of a site imports the pages and the navigation of every
declaration of that site, and removes the pages and nodes none of them
defines. The `io.meeds.administration` properties also set the mode of
this file's `global` declaration.

**Fix**: the comment says that, with `overwrite`, either property
reloads the pages and the navigation of all the administration
declarations, Meeds' included, and removes the pages and nodes none of
them defines, and that the `io.meeds.administration` property reloads
`global` as well. Comment only; `xmllint --noout` on the file at
`943e39f`: no output (well-formed).

**Release order**: merge with or after Meeds-io/portal#1338, whose
behaviour the comment describes.

Knowledge: Meeds-io/eng-standards#409, #560, #586, #587

This change is classified N3: no REST/DAO/schema/ACL surface touched (a
comment in a configuration file).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit d624df8)
@boubaker

Copy link
Copy Markdown
Member Author

Backport evidence

Source: #232, squash-merged into feature/maintenance. Its change is on origin/feature/maintenance at d624df82c9 (matched against the branch's current head, 2026-10-27). The PO validated the task on the maintenance acceptance server on 2026-10-27.

Picks: 7930f3633f = git cherry-pick -x d624df82c9, on origin/develop (786ee63943).

Adaptations: none. The pick applied cleanly, and no commit was added.

Verification

  • git show -U0 <pick> | git patch-id --stable gives the same result on 7930f3633f and on d624df82c9: 964bb695dcd6.
  • gh pr diff 232 | git patch-id --stable equals git show 7930f3633f | git patch-id --stable: 40c75d37bed5.
  • The source's approval is on its head: APPROVED by ahamdi at 943e39fcfe, which is the source PR's head.
  • mvn -B -ntp -Dcheckstyle.skip=true -pl digital-workplace-webapps -am verify at 7930f3633f: BUILD SUCCESS. The module has no tests, and the change is a comment in a configuration file.
  • Merge order: none. The change stands alone.

Level: N3, restated from the source: no REST/DAO/schema/ACL surface. A backport from feature/maintenance whose every pick's patch-id equals its source's diff merges on its first approval (ai-review-and-merge.md §3). It lands by squash, with the source's body as its message.

@github-actions github-actions Bot added the partialCIBuild Perform Partial CI Build label Oct 27, 2026
@sonarqubecloud

Copy link
Copy Markdown

@boubaker
boubaker requested review from Jihed525 and ahamdi October 27, 2026 15:24
@boubaker
boubaker enabled auto-merge (squash) October 27, 2026 15:24
@boubaker
boubaker merged commit 6d04c30 into develop Oct 27, 2026
13 checks passed
@boubaker
boubaker deleted the backport/EXO-91183-portal-configuration branch October 27, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

partialCIBuild Perform Partial CI Build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants