-
Notifications
You must be signed in to change notification settings - Fork 5
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
Group 28 - Global Cost of Living #28
Comments
Group 27 Review of Group 28: Global Cost of LivingOverall, your dashboard looks excellent. Many of the discussion points that we bring up are small details and personal preferences of ours. Personally, we think your dashboard is already very complete barring a few technical bugs that are easily fixable. PersonaProvide 3 pieces of constructive feedback that could help the app to be better suited to the scenario it was designed for
Provide 3 pieces of constructive feedback that could help the app to be better suited to the user (persona) it was designed for
FrontendAre the text (plot labels / titles / subtitles) easy to read? Are the plots selected appropriate for the data? Can you think of other alternatives or improvements for the plots regarding interactivity/reactivity or plot selection? Was the dashboard organized in a way that was easy to read (columns / rows) or you would suggest a change in this sense? Would you prefer including tabs / navbars to improve the navigation or you think it is better to visualize everything in one screen by the nature of the content? Are the plots or other images described properly for the persona described? BackendAre the options that you can select in the widgets working fine or have you found some cases where it is not working? If there are interactive plots: are the labels clear and easy to understand? DocumentationWas the documentation easy to understand from the README.md how to use the app? How could the documentation be improved? Was the app intuitive to use without reading the documentation? What information could be good to include? Indicate something you would like to include in your own app after reviewing this one: Group 27 |
Thank you Group 27 for the wonderful feedback. We really found those feedbacks very helpful and constructive and tried to implement as many as we could. Feedbacks incorporated:
Frontend
Backend
Changes not incorporated:
Once again, thank you for your valuable feedback as it helps us improve, keep them flowing in.!! |
DSCI532-peer-review
To complete by the group
proposal.md
file where it is the description of the persona and scenario: https://github.com/UBC-MDS/Cost_of_living_py/blob/v0.1.0/doc/Proposal.mdTo complete by the group reviewing:
Persona
Frontend / layout
Backend
Documentation
README.md
how to use the app? How could the documentation be improved?Indicate something you would like to include in your own app after reviewing this one
The text was updated successfully, but these errors were encountered: