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

Change url to Austin crash report dataset #1295

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Conversation

chiaberry
Copy link
Member

@chiaberry chiaberry commented Sep 25, 2023

Associated issues

cityofaustin/atd-data-tech#13068

Testing

URL to test:
https://deploy-preview-1295--atd-vzv-staging.netlify.app/

Steps to test:

The "data" link in the footer is changed to only link to the Austin Crash Report data set.

Note that I only changed the title of the dataset (since the url changes with the title) and have not changed any of the description.


Ship list

  • Run migration steps in readme in /atd-vzd/ if needed
  • Code reviewed
  • Product manager approved

@patrickm02L
Copy link
Member

I updated the description

Copy link
Member

@patrickm02L patrickm02L left a comment

Choose a reason for hiding this comment

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

Works as expected. Thanks Chia! 🚌

Copy link
Contributor

@mddilley mddilley left a comment

Choose a reason for hiding this comment

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

🙏 🚀

@chiaberry chiaberry merged commit 783392b into master Sep 25, 2023
8 checks passed
@johnclary johnclary deleted the 13068-update-footer-link branch October 5, 2023 15:23
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.

4 participants