Skip to content

AAI-281 GUI admin for the backend database (refreshed to clean up the diff)#42

Merged
marius-mather merged 33 commits into
mainfrom
db-admin
Jul 31, 2025
Merged

AAI-281 GUI admin for the backend database (refreshed to clean up the diff)#42
marius-mather merged 33 commits into
mainfrom
db-admin

Conversation

@marius-mather

Copy link
Copy Markdown
Collaborator

Description

AAI-281: implement a GUI to view/manage the AAI backend's database, using https://aminalaee.github.io/sqladmin/ .

Screenshot 2025-07-29 at 9 41 08 am

Changes

  • Set up sqladmin against our DB
  • Authenticate access to the admin using Auth0 roles (user has to have one of the roles configured in ADMIN_ROLES)
  • Add our existing DB models to the admin (view-only permission for now)
  • Unit tests of access/authentication to the DB

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)

How to Test Manually (if necessary)

Run uv run fastapi dev and visit localhost:8000/db_admin: you must have ADMIN_ROLES set to include a role like biocommons/role/biocommons/admin in .env, and your Auth0 user account must have this role.

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

Nick picking the secrets as usual

Comment thread main.py

@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 - thanks!

@marius-mather marius-mather merged commit af79977 into main Jul 31, 2025
4 checks passed
@amandazhuyilan amandazhuyilan deleted the db-admin branch June 19, 2026 03:21
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