Skip to content

feat: add eDNA Explorer platform#284

Open
amandazhuyilan wants to merge 3 commits into
mainfrom
AAI-860-add-edna-explorer
Open

feat: add eDNA Explorer platform#284
amandazhuyilan wants to merge 3 commits into
mainfrom
AAI-860-add-edna-explorer

Conversation

@amandazhuyilan

@amandazhuyilan amandazhuyilan commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

AAI-860: Adds eDNA Explorer as a platform. Unlike bpa/galaxy/sbp, access is manual-approval only (not auto-granted at registration).

Changes

  • Add EDNA_EXPLORER to PlatformEnum
  • Add eDNA Explorer to admin PLATFORM_MAPPING
  • Alembic migration adding the EDNA_EXPLORER enum value (row created by the existing populate_platforms_from_auth0 sync)
  • Not added to DEFAULT_PLATFORMS → new users land in PENDING until an admin approves
  • Tests: admin platform create + assertion that registration does not auto-grant eDNA

Related infra code changes here: https://github.com/AustralianBioCommons/aai-infrastructure/pull/32

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)
  • For any new secrets, I have updated the shared spreadsheet and the GitHub Secrets.

How to Test Manually (if necessary)

CI tests shall pass!

amandazhuyilan and others added 3 commits July 17, 2026 10:10
- Add EDNA_EXPLORER to PlatformEnum + admin PLATFORM_MAPPING
- Alembic migration adding the EDNA_EXPLORER enum value
- Keep it out of DEFAULT_PLATFORMS so access is manual-approval only
- Tests: admin create + no auto-grant on registration

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@amandazhuyilan
amandazhuyilan force-pushed the AAI-860-add-edna-explorer branch from 3579bec to b78e970 Compare July 17, 2026 00:40
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.

1 participant