-
Notifications
You must be signed in to change notification settings - Fork 13
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
Stats Page Revamp Stage 2 #250
Comments
|
Thanks for reviewing @thuongho ! @nikobooth Phase 2 is "almost" ready! Can you please review if it is as you expect? The remaining dev tasks are: I think we can show "Bay Area Average" line chart in default without placing the button. User can just select counties to compare with "Bay Area Average". "Bay Area Average" is one of the key data as we already show on the top and we do not need to let user to select or deselect in the comparison chart. Let me know your thoughts. As for "California Average" "National Average", we need to find the additional data source to support this. Is it okay to lower priority of these two? |
Oh, one important question I was about to ask is regarding the "Cases per 1,000 People" chart, did you mean it is for "Daily Cases per 1,000 People" or "Total(Cumulative) Cases per 1,000"? Right now it is calculated as "Daily Cases per 1,000 People". If you meant it should be calculated as total cases (Cumulative Cases), I will change it accordingly. |
This is looking excellent, here are my notes:
Overall, fantastic work!! We can discuss further tonight! |
UI Spec:
https://www.figma.com/file/iUS5LdAnuTKVei88QwPPCH/Bay-Area-Pandemic-Dashboard-(PanDa)-by-Bay-Area-Brigades?node-id=1286%3A5448
Sub Dev Tasks
TBD
Support "Bay Area Average" "California Average" "National Average" as there is no data available in the data scraper we use.
The text was updated successfully, but these errors were encountered: