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
fixes bug 884944 - Colors in /daily once again don't match #1419
fixes bug 884944 - Colors in /daily once again don't match #1419
Conversation
@@ -537,6 +550,11 @@ def daily(request, default_context=None): | |||
context['data_table'] = data_table | |||
context['graph_data'] = json.dumps(cadu) | |||
context['report'] = 'daily' | |||
print context['os_names'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preeent
As discussed in person, the ordering of the versions has now been fixed in peterbe@187e18b |
@@ -246,7 +246,6 @@ | |||
<div class="body"> | |||
<div id="adu-chart"></div> | |||
<p class="adu-chart-help">This graph uses an approximate <a href="https://wiki.mozilla.org/Socorro/SocorroUI/Branches_Admin#Throttle">throttle value</a> for each version, which may not be completely accurate for the entire time period.</p> | |||
<div id="adu-chart-legend"></div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove this?
r+ when Leeroy is happy. FWIW I ran the tests locally and they pass. |
Merged 9a11802 |
@brandonsavage r?
I recommend you pull down my fork and play around with it.