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

Issues to address for v3 expanded test #63

Open
10 of 18 tasks
krook opened this issue Mar 4, 2021 · 2 comments
Open
10 of 18 tasks

Issues to address for v3 expanded test #63

krook opened this issue Mar 4, 2021 · 2 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request

Comments

@krook
Copy link
Member

krook commented Mar 4, 2021

  • Abstract the notification time settings
  • Consistently use UTC in all displays (chart is currently local)
  • Make the x axis reflect time average over the full period (Update the chart to math minute by minute history, not just latest readings #62)
  • Make the y limit the red threshold, or the blue line that's currently over it
  • Make change/update indicators more prominent (debug CSS animation fade in)
  • Error handle missing data or broken websocket
  • Look into the date parsing difference between Chrome and Safari
  • Ensure mobile display responsiveness, particularly on iPad
  • Update main dashboard to show overall red/yellow/green for each gas currently in danger (in any time-window) (ppt_1)
  • Remove the graph on the default detail now view (it's the 10 min avg one) (ppt_2)
  • Add a new live tab on the details page, remove it from the time views (ppt_3)
  • Update the chart to be a line chart rather than area chart(ppt_4)
  • Add chart to the Latest tab on the firefighter's page (i.e. history of sensor readings) (ppt_5)
  • Add labels to the chart lines on firefighter pages (e.g. 10 min limit, 10 min avg, probably don't need to label the yellow line) (ppt_6)
  • Enlarge the clickable area on the Red / Yellow / Green icons on the homepage and FF pages (ppt_9)
  • Make authorization a configuration flag
  • Have the timestamp local reflect the i18n local storage language (needs to be set explicitly now)
  • Add a tooltip over the datapoints in the chart
@krook krook added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Mar 4, 2021
@krook krook self-assigned this Mar 4, 2021
@krook krook changed the title Issues for v3 expanded test Issues to address for v3 expanded test Mar 4, 2021
@JSegrave-IBM
Copy link
Collaborator

Added a few new ones - I'll slack you pics/slides for these:

  • Enlarge the clickable area on the Red / Yellow / Green icons on the homepage and FF pages (ppt_9)
  • Add chart to the Latest tab on the firefighter’s page (i.e. history of sensor readings) (ppt_5)
  • Add labels to the chart lines on firefighter pages (e.g. 10 min limit, 10 min avg, probably don’t need to label the yellow line) (ppt_6)

Changed this one

  • FROM Update main dashboard to show if any of the past readings are in danger, if not the live one
  • TO Update main dashboard to show overall red/yellow/green for each gas currently in danger (in any time-window) (ppt_1)
    I hope this is OK - it's to reflect that there's no 'past danger', per-se - all of the gases and all of averaging periods are 'live' all the time. (i.e. they're all about what's likely to be present in a FF's body 'live' right now - either from a short-sharp-shock that's only just happened or from accumulation over a longer time)

Updated this one:

@krook
Copy link
Member Author

krook commented Mar 19, 2021

@pedrocruzio these are the two bugs it would be great for you to look at:

  • Consistently use UTC in all displays (chart is currently local)
  • Look into the date parsing difference between Chrome and Safari (iPad and mobile devices, including Android)

Key files to look at are the ones that drive the dashboard (home) and details (chart) pages:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants