Skip to content

fix(processing): persist dead-lettered events - #558

Merged
Xhristin3 merged 2 commits into
XStreamRollz:mainfrom
Agaki00:issue-525-dead-letter
Aug 30, 2026
Merged

fix(processing): persist dead-lettered events#558
Xhristin3 merged 2 commits into
XStreamRollz:mainfrom
Agaki00:issue-525-dead-letter

Conversation

@Agaki00

@Agaki00 Agaki00 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Provide a short description of the changes in this PR and the motivation/why.

Related issues

Link any related issues. Use Closes #<id> to close issues automatically.

Closes:

Type of change

  • feat
  • fix
  • docs
  • chore
  • test
  • ci

Match the PR title to Conventional Commits: <type>(<scope>): <summary>.

Testing performed

Describe how you tested this change. Include commands, environment, and any setup steps.

Example:

cd api
npm run lint
npm run build
npm test

Screenshots (if applicable)

Attach screenshots or animated GIFs for UI changes.

Checklist — author

  • I rebased onto origin/main and resolved conflicts.
  • I ran the quality gates locally: npm run lint, npm run build, npm test.
  • Title follows Conventional Commits and references the issue (see above).
  • I added/updated tests where applicable and they pass locally.
  • No new TypeScript errors or lint warnings introduced.
  • I updated documentation if the change affects public behavior.
  • Screenshots included for UI changes.

Checklist — reviewer guidance

  • Required checks: lint, build, unit tests, and any package-specific E2E must be green.
  • Request review from CODEOWNER(s) for touched packages.
  • Prefer small, focused PRs. If large, confirm feature-flagging or follow-up tasks.

Follow the full PR process in CONTRIBUTING.md — particularly: sync with main, run quality gates, request a CODEOWNER review, and use squash-and-merge with a Conventional Commit-style squash message.

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

LGTM

@Xhristin3
Xhristin3 merged commit 7a6ee64 into XStreamRollz:main Aug 30, 2026
5 of 15 checks passed
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