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

Completely remove amcharts from repository if no longer in use #22

Closed
brylie opened this issue Sep 7, 2017 · 2 comments
Closed

Completely remove amcharts from repository if no longer in use #22

brylie opened this issue Sep 7, 2017 · 2 comments
Milestone
0.2.0 Roadmap

Comments

@brylie
Copy link
Collaborator

brylie commented Sep 7, 2017 •
edited
Loading

Based on discussion in #20, the amcharts library is being deprecated. If it is no longer needed, remove the amcharts directory entirely from this repository.
@brylie brylie added this to the 0.2.0 Roadmap milestone Sep 7, 2017
@brylie brylie mentioned this issue Sep 7, 2017
Closed
synw added a commit that referenced this issue Sep 7, 2017
@synw
Remove Amcharts folder from static directory #22
1a7c7f6
@synw
Copy link
Owner

synw commented Sep 7, 2017

I forgot to remove the folder from the static dir: corrected, thanks.

@synw synw closed this as completed Sep 7, 2017
@brylie
Copy link
Collaborator Author

brylie commented Sep 7, 2017

Cool, it looks like amcharts was a large portion of this codebase!

screenshot_20170907_112900

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.2.0 Roadmap
Development

No branches or pull requests
2 participants
@brylie @synw