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 admin drilldown analytics chart behavior #433

Closed
GUI opened this issue Mar 15, 2018 · 1 comment
Closed

Fix admin drilldown analytics chart behavior #433

GUI opened this issue Mar 15, 2018 · 1 comment

Comments

@GUI
Copy link
Member

GUI commented Mar 15, 2018

In the admin analytics, the drilldown chart behavior can be a bit whacky, and isn't always correct. This is something I've noticed for a while, but never quite got around to figuring out why or when it misbehaves. I finally dug into this a bit more, and figured out the data for the chart was not always being cleared each time you drilled down another layer in the URL path hierarchy. So as you drilled down, the table of data below was always correct, but the chart above could end up with left-over data from the previous level. If you refreshed the page or applied new filters, the problem would disappear, so that's why it seemed somewhat sporadic.

@GUI
Copy link
Member Author

GUI commented Mar 15, 2018

And of course this weird behavior was fixed by a simple 1 line change (to add an option to tell the charting library to replace the data on changes, rather than trying to merge it): NREL/api-umbrella@53d263b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant