-
-
Notifications
You must be signed in to change notification settings - Fork 62
Feature/640 api proxy #665
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.
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
base: main
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for luckyparking failed.
|
glenflorendo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@parcheesime I like the direction, but this can be implemented without creating a separate backend in a different tech stack.
Also, this (and other) PRs should wait to be merged until we get the build system changes in. Since I can't actively work on this regularly, this would hopefully help @bltomlin and other new developers ease into the codebase more smoothly.
| @@ -0,0 +1,59 @@ | |||
| [ | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a neat idea. We should figure out how big (how many records) we want this to have. It should be a big enough sample of data.
| @@ -0,0 +1,48 @@ | |||
| # Issue #640 ('https://github.com/hackforla/lucky-parking/issues/640') | |||
|
|
|||
| This FastAPI app allows Lucky Parking developers to TEST features while the main data pipeline is under construction. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious why a different tech stack and a different server?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to clarify — this proxy is meant to be strictly temporary and for local development/testing only, especially to unblock development while the main data pipeline is in progress.
It’s not intended to be deployed nor to become part of the final architecture. Once the data pipeline is ready, this should be removed or replaced.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re: mock data — also note that the static JSON is intentionally small and controlled
right now to support lightweight map tests and local builds.
I'm happy to adjust anything needed to move things forward — just let me know what you prefer. Thanks, Glenn!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious why a different tech stack and a different server?
I’m happy to rewrite this to match the existing stack.
Description
This PR adds a temporary FastAPI proxy to support development while the data
pipeline is still in progress.
/citationsreturns parking citation data from either a static mock file or thelive LA Open Data API.
/pingconfirms mode status and checks if the live API is reachable..envfile viaUSE_STATIC_DATA=true/false.Includes a
README.mdwith setup and usage instructions.Related Issues
Closes #640
Testing
.envbetween true/false/citations,/ping) respond correctly