Skip to content

Initial database support - #226

Open
benson31 wants to merge 26 commits into
FLASK-LLNL:mainfrom
benson31:project-crud
Open

Initial database support#226
benson31 wants to merge 26 commits into
FLASK-LLNL:mainfrom
benson31:project-crud

Conversation

@benson31

@benson31 benson31 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

This PR includes a bunch of FastAPI async endpoints for basic CRUD database operations for project management. Database interactions are managed through SQLAlchemy ORM, and migrations are managed with Alembic. The aim was minimal changes to the frontend, and that appears to be largely successful (except, of course, for async/await contamination).

Outstanding TODOs

  • Amend pyproject.toml to reflect new dependencies.
  • Support auto-migration from LocalStorage to the database.
  • MariaDB for deployment.
  • New user registration. (in deployment; locally users will just be 'nobody')
  • A plethora of FIXMEs left in the code.

@benson31
benson31 requested review from bvanessen and tbennun July 20, 2026 14:09
@benson31 benson31 self-assigned this Jul 20, 2026
@benson31 benson31 added the enhancement New feature or request label Jul 20, 2026
@benson31
benson31 marked this pull request as ready for review August 6, 2026 15:15
Comment thread charge_backend/database/routes/experiment.py
Comment thread charge_backend/database/routes/local.py
Comment thread charge_backend/database/routes/project.py Outdated
Comment thread charge_backend/database/settings.py Outdated
Comment thread charge_backend/tests/routes/conftest.py
Comment thread flask-app/src/App.tsx Outdated
@benson31
benson31 force-pushed the project-crud branch 3 times, most recently from a9fcc49 to 63e48c0 Compare August 13, 2026 16:50
benson31 and others added 18 commits August 24, 2026 09:09
Includes a bunch of FastAPI endpoints for basic CRUD database
operations for project management. Database interactions are managed
through SQLAlchemy ORM, and migrations are managed with Alembic. The
aim was minimal changes to the frontend, and that appears to be
largely successful (except, of course, for async/await contamination).
Co-authored-by: Tal Ben-Nun <tbennun@users.noreply.github.com>
Co-authored-by: Tal Ben-Nun <tbennun@users.noreply.github.com>
@benson31
benson31 force-pushed the project-crud branch 4 times, most recently from b796682 to 109d161 Compare August 24, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants