Serco627 / workout-app

https://workout-app-git-main-sercos-projects.vercel.app/
4 stars 1 forks source link

User Story: Use Local Storage State #18

Closed anapecic closed 1 month ago

anapecic commented 2 months ago

Value proposition

As a User

I want to keep my workots saved even when I refresh the page,

So that I don´t lose my progress or have to recreate my workout list every time I revisit the app.

Description

Instead of using stats that store created or deleted workouts as along as a browser session is during, the local storage is installed

Acceptance criteria

Tasks

ahohnsen commented 1 month ago

Hi!

What you’re aiming to achieve here is to ensure that the workouts are persisted and not lost after a page refresh. I would adjust the Value Proposition accordingly, and also the Acceptance Criteria, as they should not include technical details.

Value Proposition

As a user

I want to keep my workouts saved even when I refresh the page,

so that I don’t lose my progress or have to recreate my workout list every time I revisit the app.

Acceptance Criteria

Hope, that helps. 🙂

LorenzCoachK commented 1 month ago

Hi Andrea, thank for the precise feedback in this specific user story. I had some difficulties in finding the wording for this technic topic. Thanks a lot for your help! It shows me exactly what the benefit is from the users perspective.

Merci beaucoup! ☺️

LorenzCoachK commented 1 month ago

Comment from Andrea:

Everything seems to be working fine. However, when checking the local storage in my browser, I noticed that you’re saving the exercise sets and reps of a new workout as strings. This is not consistent with the existing data, where the reps and sets are saved as numbers. I strongly recommend keeping it consistent by saving the reps and sets only as numbers. As the fix is only a minor change in your code, you do not need to post the update again for a code review and quality assurance.