Migrate web backend to FastAPI #210
Conversation
shiv-tyagi
force-pushed
the
fix-apis
branch
2 times, most recently
from
372e035 to
58c1553
Compare
shiv-tyagi
force-pushed
the
fix-apis
branch
4 times, most recently
from
974e81d to
4d1ff84
Compare
|
This is running on custom-beta and working good. |
|
Hey @peterbarker @Williangalvani! Did you get a chance to review this? |
|
This is also a problem on the current custom build server. Build Cube Orange Plus with default parameters fails: Vehicle ID: copter
|
Williangalvani
left a comment
Williangalvani
left a comment
There was a problem hiding this comment.
I spent a while looking through this and couldn't find anything outstanding.
Using an "Admin token" is a bit weird to me, but other than that, it looks fine.
I built a firmware locally with no issue.
@Williangalvani You mean it could have been something with lesser scope? |
3 participants
This is a big PR which migrates our backend to the modern FAST API. Our existing Flask based code was cluttered and had made it very less extensible. The new fastapi based backend makes it very structured and modular.
Now our all endpoints are as per standard REST API guidelines.
The existing Flask backend was also very tricky to write end to end tests. The new backend structure also makes things easily testable and tests would follow this PR.
It also provides a self generated api documentation endpoint (/api/docs) which would help in integrating custom build server with external services like ground control stations in future.
