Skip to content

Dev - #120

Closed
jumpy-cat wants to merge 22 commits into
mainfrom
dev
Closed

Dev#120
jumpy-cat wants to merge 22 commits into
mainfrom
dev

Conversation

@jumpy-cat

Copy link
Copy Markdown
Member

Need the CI setup in main for branch protections to work properly

@jumpy-cat jumpy-cat closed this Oct 21, 2026
@jumpy-cat jumpy-cat reopened this Oct 21, 2026
@jumpy-cat
jumpy-cat marked this pull request as draft October 26, 2026 19:54
jumpy-cat and others added 6 commits November 3, 2026 12:07
* Use unix prediction instead of converting based on predicted minutes

* fix(ci/cd): Update Typecheck command to use npx

* test(notif): enforce presence of ride predictions

* remove stale comment

---------

Co-authored-by: Edward Zhang <edwzhang@umich.edu>
* feat(api): add turn by turn directions types

Replaces the raw turn angle format with enum-typed steps that map
directly to the instructions the frontend shows. Types only; mock
data comes in a follow-up PR.

Part of #97.

* test(api): add unit tests for turn by turn directions types

* refactor(api): move shared direction step fields out of the union

swagger_parser doesn't extract common fields from a discriminated
union, so the shared fields now live on DirectionStep and the
variants are in a separate `instruction` field that only shares `type`.

* fix(api): add openapi discriminator to direction instructions

Without it, swagger_parser's generated Dart client tries each variant
in order and decodes every instruction as DepartInstruction, since that
variant only requires `type`. With the discriminator it switches on
`type` instead.
…ity (#139)

* Removed dotenv from project

* Remove accidental allowScripts addition and reorganize

* remove loadEnvFile() and corresponding import statements because they actually aren't needed. flags will autopopulate everything before program runs.

* remove loadEnvFile() and corresponding import statements because they actually aren't needed. flags will autopopulate everything before program runs.
@jumpy-cat
jumpy-cat marked this pull request as ready for review November 14, 2026 03:13
@jumpy-cat jumpy-cat closed this Nov 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants