Skip to content

CELDEV-1334 Add navigation REST API - #309

Open
fpichler wants to merge 4 commits into
devfrom
feature/CELDEV-1334
Open

CELDEV-1334 Add navigation REST API#309
fpichler wants to merge 4 commits into
devfrom
feature/CELDEV-1334

Conversation

@fpichler

@fpichler fpichler commented Jul 31, 2026

Copy link
Copy Markdown
Member

https://synjira.atlassian.net/browse/CELDEV-1334

Summary

  • add celements-navigation-rest with the public /api/v1/navigation/{nodeSpace} contract
  • enforce current-wiki canonical references, caller-relative rights filtering, localized titles and URLs, stable segmented JSON, and safe errors
  • expose OpenAPI documentation and apply Cache-Control: private, no-store to successful and error responses
  • cover request validation, guest/authenticated rights behavior, expansion, ordering, mandatory-value omission, JSON, errors, and OpenAPI

Coordinated delivery

Draft 2 of 4 for CELDEV-1334:

  1. CELDEV-1334 Add current-wiki REST reference ADR celements-spring#31
  2. CELDEV-1334 Add navigation REST API #309
  3. CELDEV-1334 Manage navigation REST artifact celements-parent-poms#142
  4. CELDEV-1334 Deploy navigation REST component celements-web#556

Validation

  • offline clean verify: 43 tests, zero failures/errors
  • Maven dependency analysis: no problems
  • formatter, diff check, and fully-qualified-name scan: clean
  • packaged REST JAR matched the copy embedded in the coordinated webapp WAR
  • local guest and error-contract runtime smoke checks passed
  • manually tested and approved by the requester

@fpichler
fpichler requested a review from msladek July 31, 2026 22:59
@fpichler
fpichler marked this pull request as ready for review July 31, 2026 22:59
@msladek msladek assigned fpichler and unassigned msladek Aug 2, 2026
@fpichler
fpichler requested a review from msladek August 4, 2026 23:09
@fpichler fpichler assigned msladek and unassigned fpichler Aug 4, 2026

@msladek msladek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for the changes, lgtm

@msladek msladek assigned fpichler and unassigned msladek Aug 5, 2026
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