Skip to content

PP-14739 Fix HTTP 409 response when inviting user to a service - #5037

Merged
felixmcd-mt merged 1 commit into
masterfrom
PP-14739-fix_selfservice_409
Sep 11, 2026
Merged

felixmcd-mt merged 1 commit into
masterfrom
PP-14739-fix_selfservice_409

Conversation

@felixmcd-mt

@felixmcd-mt felixmcd-mt commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

What

PP-14739

A brief description of the pull request

  • Handle 409 response error in invite.controller.js
  • Add unit tests for 409 response handling in invite.controller.test.js
  • Format changed files with Prettier

How

Steps to test or reproduce

  • In Selfservice, invite any email address to a service

  • As a different user on the same service, send an invite to the same email address

Screenshots (views have been added/changed)

Screenshot 2026-09-11 at 15 31 10

@felixmcd-mt
felixmcd-mt force-pushed the PP-14739-fix_selfservice_409 branch from 124d754 to d67950a Compare September 11, 2026 14:09
@felixmcd-mt
felixmcd-mt marked this pull request as ready for review September 11, 2026 14:59
- Handle 409 response error in invite.controller.js
- Add unit tests for 409 response handling in invite.controller.test.js
- Format changed files with Prettier
@felixmcd-mt
felixmcd-mt force-pushed the PP-14739-fix_selfservice_409 branch from d67950a to 1382ae1 Compare September 11, 2026 15:16
@felixmcd-mt felixmcd-mt changed the title PP-14739 Fix Selfservice not handling HTTP 409 response when inviting… PP-14739 Fix HTTP 409 response when inviting user to a service Sep 11, 2026

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

Good work, Felix! 👍

@felixmcd-mt
felixmcd-mt merged commit 63c6afa into master Sep 11, 2026
15 checks passed
@felixmcd-mt
felixmcd-mt deleted the PP-14739-fix_selfservice_409 branch September 11, 2026 15:34
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