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

Adjusting selected poi component #55

Closed
wants to merge 5 commits into from

Conversation

emily-yarvis
Copy link
Collaborator

Developer: Emily Yarvis

Closes #53, #51, #48 (partially)

Pull Request Summary

  • Renaming some files and reorganizing to make the repo clearer
  • Fixing issue with progress to ensure you could complete POI's out of order and the correct progress would be stored regardless
  • Fixing issue with progress state so it is now saved within each session

Modifications

  • Renamed component files to make them more clear and moved the styling files to the styling folders
  • Adjusted the POI list component file to allow for users to complete issues out of order and the progress bar reflecting this
  • Adjusting the POI list component file such that it uses sessionStorage to save the progress state so that users can switch pages within the web application with their progress being saved within their session.

Testing Considerations

Dev deployed to test

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

{put screenshots of your change, or even better a screencast displaying the functionality}

Firstly this changes adjusts the functionality of the progress bar such
that when you complete one POI you dont nessacarily complete the POI's
before it since they are NOT in order. Additionally I adjusted the
selected poi component to take in the current progress rather then being
hard coded to an order.
Currently when you click on a POI and then return to the list view page,
your progress is reset to zero. This change fixes that by saving the
stat of your progress within your session. Saving within yur session
Copy link

vercel bot commented Feb 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lcslo-kco ❌ Failed (Inspect) Feb 20, 2025 4:36am

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

Successfully merging this pull request may close these issues.

Bug With Progress Tracking
1 participant