Skip to content

AAI-384: Registration endpoint for SBP#74

Merged
minh-biocommons merged 5 commits into
mainfrom
sbp-register
Sep 23, 2025
Merged

AAI-384: Registration endpoint for SBP#74
minh-biocommons merged 5 commits into
mainfrom
sbp-register

Conversation

@minh-biocommons

@minh-biocommons minh-biocommons commented Sep 21, 2025

Copy link
Copy Markdown
Collaborator

Description

AAI-384: Registration endpoint for SBP

Changes

  • Added registration endpoint for SBP
  • Updated relevant platform mappings & schemas
  • Added/updated relevant tests

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)

- Introduced SBP registration request schema.
- Implemented SBP user registration endpoint with email notification.
- Updated platform enums and mappings to include SBP.
- Added tests for SBP registration, including validation and error handling.
amandazhuyilan
amandazhuyilan previously approved these changes Sep 22, 2025

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

Looks great - just again nit picking on small stuff...

Comment thread routers/sbp_register.py Outdated
Comment thread routers/sbp_register.py Outdated
marius-mather
marius-mather previously approved these changes Sep 22, 2025

@marius-mather marius-mather left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, just wanted to check whether SBP platform membership should be auto-approved or not.

Comment thread routers/sbp_register.py

@marius-mather marius-mather left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go

@minh-biocommons minh-biocommons merged commit 92fa026 into main Sep 23, 2025
2 checks passed
@amandazhuyilan amandazhuyilan deleted the sbp-register branch November 24, 2025 22:23
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.

3 participants