Skip to content

Fixed - #482

Open
69starman wants to merge 4 commits into
mergepay:mainfrom
69starman:fixed
Open

Fixed#482
69starman wants to merge 4 commits into
mergepay:mainfrom
69starman:fixed

Conversation

@69starman

Copy link
Copy Markdown

CLOSES #328

What does this PR do?

Checklist

  • npm run build passes
  • npm test passes (offline — DB/network mocked)
  • New endpoints validated with Zod + permission checks
  • Audit logs added for state-changing actions
  • Contract changes mirrored in mergepay-web
  • No secrets committed

Related issues

@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@69starman Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@mergekeeper

mergekeeper Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Needs review

Linked to #328, but the diff does not match the issue scope.

The pull request introduces changes for OpenAPI plugins, markdown fix docs, and unrelated route validations instead of implementing the requested audit logging middleware and service for group expense creation and debt settlement.

Reviewed commit: a50a1a8012df110fe1b498c14b4cc8a1ee23e065.

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.

Implement audit logging middleware for group expense creation and settlement actions

1 participant