Skip to content
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

Route color #478

Merged
merged 7 commits into from
Mar 7, 2016
Merged

Route color #478

merged 7 commits into from
Mar 7, 2016

Conversation

doublestranded
Copy link
Contributor

Closes #468

  • default color for routes is nil
  • includes color attribute in RouteStopPattern API query

# Copy over GTFS attributes to tags
route.tags ||= {}
route.tags[:route_long_name] = entity.route_long_name
route.tags[:route_desc] = entity.route_desc
route.tags[:route_url] = entity.route_url
route.tags[:route_color] = entity.route_color
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should keep this for now for backwards compatibility.

@drewda
Copy link
Member

drewda commented Mar 5, 2016

@doublestranded looks like some merge conflicts to resolve... but those can certainly wait 'til Monday.

drewda added a commit that referenced this pull request Mar 7, 2016
@drewda drewda merged commit 2fa1294 into master Mar 7, 2016
@drewda drewda deleted the route_color branch March 7, 2016 19:51
@drewda drewda removed the in progress label Mar 7, 2016
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.

None yet

2 participants