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

Add prefix to api #14

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Add prefix to api #14

merged 2 commits into from
Sep 25, 2024

Conversation

cscollett
Copy link
Contributor

To mimic current production paths I've added a prefix of "api" to the Flask blueprint routing.

@cscollett
Add a prefix to the API to mimic current functionality
f49c0c9
@cscollett
Update dependencies while rebuilding
80f263c 
Updating minor dependencies and major Gunicorn update. Leaving mysql connector to be updated at a later patch level.
RvanB
RvanB approved these changes Sep 25, 2024
View reviewed changes
Copy link
Contributor

@RvanB RvanB left a comment

Choose a reason for hiding this comment

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

Looks good to me! Nice that you can add a url prefix to the blueprint rather than change all the routes.

lampham789
lampham789 approved these changes Sep 25, 2024
View reviewed changes
Copy link
Contributor

@lampham789 lampham789 left a comment

Choose a reason for hiding this comment

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

Beauty

@cscollett cscollett merged commit 3969c56 into main Sep 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lampham789 lampham789 lampham789 approved these changes

@RvanB RvanB RvanB approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cscollett @RvanB @lampham789