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

fix: switch out SGID-based UTA data for UTA's portal #357

Merged
merged 6 commits into from
Aug 26, 2024
Merged

fix: switch out SGID-based UTA data for UTA's portal #357

merged 6 commits into from
Aug 26, 2024

Conversation

stdavis
Copy link
Member

@stdavis stdavis commented Aug 19, 2024

Ref: agrc/porter#331, agrc/porter#332, agrc/porter#334,
agrc/porter#333

@ZachBeck: I just pushed an update to https://locate.dev.utah.gov/ that switches out the UTA data from SGID with their portal data. Would you mind taking a look and making sure that I didn't break anything? I've got to run but the deploy should be done in the next 10 minutes. Thanks!

  • publish a new map service and check the layer ids before these changes are deployed

Copy link
Member

@steveoh steveoh left a comment

Choose a reason for hiding this comment

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

technical review looks good.

Is it normal that light rail is bundled into commuter rail?

@stdavis
Copy link
Member Author

stdavis commented Aug 20, 2024

Is it normal that light rail is bundled into commuter rail?

Yes, it's been that way since the beginning, IIRC.

@ZachBeck
Copy link
Member

I don't remember what the old version was like. Should the stops have more info than just the color of the trax line?

@stdavis
Copy link
Member Author

stdavis commented Aug 26, 2024

Here's a link to the current app: https://locate.utah.gov/

But it sounds like you may not be any more familiar with the UTA data in this app that I am. I fixed the popups for the stations so that they show the station name rather than the route name. I'm going to call this good. Thanks for catching that.

@stdavis stdavis enabled auto-merge (rebase) August 26, 2024 15:31
@stdavis stdavis merged commit e5a7e8a into main Aug 26, 2024
6 checks passed
@stdavis stdavis deleted the dev branch August 26, 2024 15:36
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.

3 participants