Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FastAPI migration #803

Merged
merged 77 commits into from
Sep 24, 2024
Merged

FastAPI migration #803

merged 77 commits into from
Sep 24, 2024

Conversation

erikguntner
Copy link
Collaborator

Closes #788

What changes did you make?

  • Scaffold a FastAPI project and port over authentication routes
  • Configure frontend to work with FastAPI backend
  • Update Readme
  • Rename folders

erikguntner and others added 30 commits September 5, 2024 22:25
@erikguntner
Initialize new fastapi project
cd1a72b
@erikguntner
Add settings config and db
28cf2f3
@erikguntner
Add models, db seed operation, and api router
6393cf7
@erikguntner
Add models, db see operation, and api router
c37e7d1
@erikguntner
Add auth router, dependencies, and schemas
9107b10
@erikguntner
Create crud operations and auth routes
4b32985
@erikguntner
Update README
0ad2bc8
@erikguntner
Merge pull request #787 from hackforla/fastpi-setup
3e9c298 
Fastpi setup
@erikguntner
Update README.md
0ff1d0c
@erikguntner
Update README.md
3aea3e2
@erikguntner
Update README.md
d8319bf
@erikguntner
Update README.md
cfaed98
@erikguntner
Update README.md
d84c2cd
@erikguntner
Update README.md
061b0fc
@erikguntner
Update README.md
946ee10
@erikguntner
Update README.md
32ddd8d
@erikguntner
Update README.md
feaef56
@paulespinosa
fastapi-migration: poetry for dependency mgmt only
73f5ee5
@paulespinosa
fastapi-migration: Moved tests and test config
ba0fa32 
This commit moved the API `tests` folder and configured poetry with a
test dependency group. `tox` was added along with configuration to run
`poetry run pytest`.

The test dependencies are installed using poetry:
`poetry install --with test`.
@paulespinosa
Merge remote-tracking branch 'origin/fastapi-migration' into fastapi-…
7cd84a4 
…migration-pe
@erikguntner
Fix package name
2c83714
@erikguntner
Add session routes and schemas
10c9048
@erikguntner
Removed unused import
a0a6afd
@erikguntner
Formattin
0682d0f
@erikguntner
Add /user route
aeede11
@erikguntner
Create user router
b57c098
@erikguntner
Refactor env config and remove python-dotenv package
860553a
@paulespinosa
fastapi-migration: Organize by workflow sections
5253cb6 
This commit is part of the migration from connexion to FastAPI.
The commit contains a directory structure based on workflow responsibilities.

Tests still need to be updated along with code refactoring.
Files have been moved to their respective areas of responsibilities but have not
been refactored to work within.
@johnwroge
Added forgot password route handler in api/routes/auth.py, created fo…
e8e96a7 
…rgot password response and request classes, and reformatted the calculate secret hash function used in previous api files
@johnwroge
Imported boto3
3345914
paulespinosa and others added 29 commits September 12, 2024 17:02
@paulespinosa
fastapi-migration: Document API naming convention in README.
f842cf3
@erikguntner
Merge branch 'fastapi-migration' of https://github.com/hackforla/Home…
8dcac18 
…UniteUs into fastapi-auth-flow
@erikguntner
Merge and move relationship management folder
3a75d0a
@paulespinosa
Merge pull request #793 from hackforla/fastapi-auth-flow
75de0d7 
Fastapi auth flow
@paulespinosa
fastapi-migration: Test authentication
bc3484c 
Updated and rearranged dependencies, fixed relationship management spelling
error, rewrote test_authentication.py.
@paulespinosa
fastapi-migration: Delete test_mocking
46eb2cf 
These tests aren't needed anymore.
@erikguntner
Update import
28f2f66
@erikguntner
Update confim sign up route to work with FastAPI
a3e3741
@erikguntner
Update sign up form on frontend
77837b0
@erikguntner
Change isLoading variable and Remove Google auth button
12cfac2
@erikguntner
Throw error if no type is provided
4f041ac
@erikguntner
Add signout route and remove session cookies
4b1f8d7
@erikguntner
Comment out divider from auth forms
8885528
@erikguntner
Update imports
daed4f5
@erikguntner
Change session route hash to use username, remove username from /sign…
7fc5753 
…up/confirm params
@erikguntner
Add route descriptions, protect signout route using bearer token
b57d9b4
@erikguntner
rename forgot password routes
7a5e434
@erikguntner
Change forgot password paths on frontend
a79198c
@erikguntner
Update forgot password routes responses
8d87d82
@erikguntner
Protect user route
f9449cf
@erikguntner
Add get user by id operation
c30492d
@erikguntner
change user controller to users and update paths
c60369c
@erikguntner
Update secret hash cal to use username
9c3fc45
@erikguntner
update current user query and hooks on frontend
3ea183d
@erikguntner
Fix delete user route
2e02107
@erikguntner
Get client id from settings instead of query params
1faab11
@erikguntner
Merge pull request #799 from hackforla/fastapi-auth-continued
caebdd6 
FastAPI authentication and user updates
@erikguntner
update backend .env.example
d411e5a
@erikguntner
Change folder names
6055826
@erikguntner erikguntner merged commit 71c9270 into main Sep 24, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Initial Scaffold for FastAPI Implementation
3 participants
@erikguntner @paulespinosa @johnwroge