feat(api): add clients generated from openapi specs; refactor: move busrouteline computation to backend - #105
Draft
jumpy-cat wants to merge 3 commits into
Draft
feat(api): add clients generated from openapi specs; refactor: move busrouteline computation to backend#105jumpy-cat wants to merge 3 commits into
jumpy-cat wants to merge 3 commits into
Conversation
jumpy-cat
force-pushed
the
generated-clients
branch
from
August 5, 2026 17:41
697eb4c to
72f284b
Compare
jumpy-cat
force-pushed
the
generated-clients
branch
from
August 5, 2026 17:42
72f284b to
5470bcb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Been working on end-to-end typesafety between the frontend and the backend, this PR adds the beginnings of that and trials it on the
getAllRoutesandgetAllRideRoutesroutes that I needed to change anyway (moving the logic to the backend, also in this PR, see #98).To run the frontend you will need to define an extra environment variable
BACKEND_OPENAPI_URL, which would have a value similar to what you'd setBACKEND_URLto but without the extra/mbus/api/v3at the end. Note that the url would have to point at a backend that matches the openapi spec (as of right now neither the production backend nor the mock backend, you'd have to run thenav/plan-journey-bus-polylinesbackend yourself). You will also have to generate the clients first, the ci file should have an up to date version of the commands needed.It should look something like this (full clean & regenerate):
Type of Change
feat)fix)Related Issues
Prereq of how I'd like to do #98
Depends on mbusdev/mbus-backend-dev#47
Changes Made
fetchRoutes, also remove the parsing logic as thats in the backend nowfromBackendmethods to the model types that need itflutter analyzeto check for errors (WIP, should be adjusted to match how the other CI work is doing things)Testing Done
Flutter:
Screenshots / Demo (if UI or notification change)
routes still show up! (using locally running backend)

Checklist
[type](scope): short descriptionmainand is our current working update branch (e.g.maizebus2.1)print()/debugPrint()/console.log()left in production code