Skip to content

CELDEV-1332 - Add the shared messages component and web overlay - #101

Open
fpichler wants to merge 5 commits into
devfrom
CELDEV-1332-messages
Open

CELDEV-1332 - Add the shared messages component and web overlay#101
fpichler wants to merge 5 commits into
devfrom
CELDEV-1332-messages

Conversation

@fpichler

@fpichler fpichler commented Aug 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Add the celements-messages component JAR and celements-messages-web WAR overlay.
  • Centralize flat lexical Velocity-fragment discovery and atomic JSON rendering in MessageService.
  • Expose the same aggregation through the REST controller and $services.celementsMessages, leaving only one shared deprecated adapter pair.

Validation

  • 15 focused controller, service, and script-service tests
  • Component and web-overlay clean installs
  • Dependency-tree, artifact-content, adapter, fragment, and diff checks

Comment thread celements-messages/component/pom.xml Outdated
Comment thread celements-messages/web-module/pom.xml Outdated
@msladek msladek assigned fpichler and unassigned msladek Aug 2, 2026

@cbspichler cbspichler 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.

tests need fixing as already mentioned, apart from that lgtm

@cbspichler cbspichler removed their assignment Aug 3, 2026
@fpichler
fpichler requested a review from msladek August 4, 2026 05:09
@msladek msladek assigned msladek and fpichler and unassigned fpichler and msladek Aug 4, 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.

3 participants