feat: Digest mail notifications - eXIP7.3.0.22 - integration into feature/mips - #495
Open
azayati wants to merge 3 commits into
Open
feat: Digest mail notifications - eXIP7.3.0.22 - integration into feature/mips#495azayati wants to merge 3 commits into
azayati wants to merge 3 commits into
Conversation
azayati
marked this pull request as ready for review
September 10, 2026 09:55
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
eXIP 7.3.0.22 — Digest mail notifications · integration into
feature/mipsThe
processcategory (new request, request commented) and its line plugin, written from scratch, and the removal of the legacymakeDigestoverrides.The 3 commit(s) replayed from
feature/experience40df5640feat: Declare the Process digest mail notifications category - EXO-89484 (feat: Declare the Process digest mail notifications category - EXO-89484 - eXIP7.3.0.22 #491)fa6ce173feat: Digest email lines for request notifications - EXO-89486_EXO-89487 (feat: Digest email lines for request notifications - EXO-89486_EXO-89487 - eXIP7.3.0.22 #493)762fc058fix: Remove the legacy digest stubs of the processes notification providers - EXO-90072 (fix: Remove the legacy digest stubs of the processes notification providers - EXO-90072 - eXIP7.3.0.22 #494)14 files changed, 354 insertions(+), 20 deletions(-)
How this branch was built
exip-7.3.0.22-mipsstarts fromorigin/feature/mipsand replays only the commits taggedeXIP7.3.0.22(git cherry-pick -x, the original reference is in every message). The FB version-bump commit (Task-87990) and the commits of the other eXIPs present onfeature/experienceare deliberately left out — nopom.xmlis touched.Merge order
makeDigestis still an abstract method ofcommons-apionfeature/mips; the other 18 PRs remove their overrides, so they only compile once commons is merged: commons#786 first (wait for the Nexus snapshot), then social#6093, then the 17 addons in any order.Classification
N1 for the whole eXIP (Liquibase schema and JPA entities, the commons notification dispatcher, new REST endpoints, mass email sending). Approver ≠ author: this PR must be approved by an Architect / Senior Developer who knows it is N1, not on the AI review alone.
Already validated on
feature/experienceEvery US of the board (project 8372) is "Tested & Validated" by the PO, the legacy engine cleanup (EXO-90072, 19 repositories) included. Full functional test plan: capture, daily and weekly contents, timezones, catch-up at startup, safety cleanup, and non-regression of the instant notifications.
Knowledge: TODO — eng-standards PR to open (
/domain-doccommons + social) before leaving draft🤖 Generated with Claude Code