Skip to content

feat: Declare the News digest mail notifications category - EXO-89484 - eXIP7.3.0.22 - #897

Merged
azayati merged 2 commits into
feature/experiencefrom
exip-7.3.0.22-us03-digest-category
Sep 1, 2026
Merged

feat: Declare the News digest mail notifications category - EXO-89484 - eXIP7.3.0.22#897
azayati merged 2 commits into
feature/experiencefrom
exip-7.3.0.22-us03-digest-category

Conversation

@azayati

@azayati azayati commented Aug 28, 2026

Copy link
Copy Markdown
Member

Part of eXIP 7.3.0.22 — Digest Mail Notifications. Task EXO-89484 (US03).

Contributes the News category to the digest settings drawer. Depends on Meeds-io/commons#772, which brings the Kernel DigestCategoryRegistry this configuration targets — merge that one first.

Pure configuration, no Java:

  • The category is declared as a component-plugin on the registry, next to the notification plugins it already declares, and covers the 3 news plugins (post, mention, publish).
  • Its label is added to the resource bundle those plugins already use, where social's DigestCategoryLabelResolver reads it — translated server side, so the settings page has nothing to load.

The categories are declared in the Kernel rather than as Spring beans on purpose: each webapp has its own Spring context, and only the Kernel container is shared by all of them. This addon's category therefore reaches the digest whatever the addon is made of.

AI contribution

Classified N1 — human-driven, no auto-merge, author ≠ approver.

azayati and others added 2 commits September 1, 2026 08:57
…- eXIP7.3.0.22

Contributes the News category to the digest settings drawer, next to
the notification plugins it covers. Pure configuration: the category is
declared on the Kernel DigestCategoryRegistry of commons, and its label lives
in the bundle the notification plugins of this addon already use.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ies - EXO-89484 - eXIP7.3.0.22

The category declarations covered every notification of their group, but the
functional spec lists exactly which notifications the digest considers: only
those 12 plugins are declared, nothing else enters a digest.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@azayati
azayati force-pushed the exip-7.3.0.22-us03-digest-category branch from 5b0aaef to 7e620f6 Compare September 1, 2026 07:57
@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@azayati
azayati merged commit f78fe6c into feature/experience Sep 1, 2026
4 checks passed
@azayati
azayati deleted the exip-7.3.0.22-us03-digest-category branch September 1, 2026 10:12
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.

1 participant