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.

Sign up for GitHub

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

Jump to bottom

#144 Data Visualization #154

Merged
merged 8 commits into from
Apr 5, 2022
Merged

#144 Data Visualization #154

merged 8 commits into from
Apr 5, 2022

Conversation

aswallace
Copy link
Collaborator

Migrate Data Voyager 2.0 from whyis

  • Linked to templated chart view
  • Added dependency style-loader for webpack to handle Data Voyager's font-awesome styling

TD:

  • Chart URIs need to be updated (also for gallery and chart view)
  • Add option to save charts directly from DV
  • Discuss how to link to other querying methods (e.g., parameterized queries)

close #144

@aswallace aswallace self-assigned this Mar 31, 2022
tholulomo
tholulomo approved these changes Apr 5, 2022
View reviewed changes
Copy link
Contributor

@tholulomo tholulomo left a comment

Choose a reason for hiding this comment

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

#accepted

@aswallace aswallace merged commit c8d5daa into develop Apr 5, 2022
@aswallace aswallace deleted the #144_Data_Visualization branch April 5, 2022 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tholulomo tholulomo tholulomo approved these changes

@stouffers stouffers Awaiting requested review from stouffers

Assignees

@aswallace aswallace

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Data Visualization
2 participants
@aswallace @tholulomo