Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

Add route description #147

Merged
merged 3 commits into from
Mar 26, 2024
Merged

Add route description #147

merged 3 commits into from
Mar 26, 2024

Conversation

OxygenCobalt
Copy link
Collaborator

Add a route description field to the database and API, rather than hard-coding it into the app. The first line in the route KML polygon is now considered the description, followed by the stops.

@OxygenCobalt OxygenCobalt requested review from a team March 24, 2024 18:50
@OxygenCobalt OxygenCobalt mentioned this pull request Mar 24, 2024
@OxygenCobalt OxygenCobalt changed the title Route description Add route description Mar 24, 2024
Copy link
Member

@ezrichards ezrichards 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. Alembic upgrade head shouldn't have any unintentional side effects when we run it this time in prod, right?

@OxygenCobalt
Copy link
Collaborator Author

I hope so @ezrichards.

@OxygenCobalt OxygenCobalt merged commit 42d708c into main Mar 26, 2024
3 checks passed
@OxygenCobalt OxygenCobalt deleted the route-description branch March 26, 2024 19:40
OxygenCobalt added a commit that referenced this pull request Apr 18, 2024
* backend: add route description field

* backend: add description to routes response

* backend: parse description from route kml
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants