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

adds fix to delete unnecessary columns #940

Merged
merged 6 commits into from
Oct 22, 2020
Merged

Conversation

tillyw
Copy link
Contributor

@tillyw tillyw commented Oct 21, 2020

Fixes cityofaustin/atd-data-tech#4163 once and for all

Parses out nested objects to create CSV with desired columns and order (as @mateoclarke originally suggested)

Examples CSV:

atd_txdot_crashes2020-10-21T19_23_58-05_00.csv.zip

Copy link
Contributor

@mateoclarke mateoclarke left a comment

Choose a reason for hiding this comment

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

Looks great, just that one suggestion to make the includes less brittle.

pedro

atd-vze/src/Components/GridExportData.js Outdated Show resolved Hide resolved
@tillyw tillyw merged commit a49f199 into master Oct 22, 2020
@tillyw tillyw deleted the 4163_update_export_csv branch October 22, 2020 16:51
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.

Update CR3 download export file
2 participants