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

Bug: Saving New Project Issues. #1710

Closed
Jonathanko52 opened this issue May 23, 2024 · 0 comments · Fixed by #1732
Closed

Bug: Saving New Project Issues. #1710

Jonathanko52 opened this issue May 23, 2024 · 0 comments · Fixed by #1732
Assignees
Labels
bug Release Note: Shows as Error Correction level: medium p-feature: Save Button pbv: dev All engineering issues priority: MUST HAVE role: front-end Front End Developer size: 1pt Can be done in 4-6 hours
Milestone

Comments

@Jonathanko52
Copy link
Member

Jonathanko52 commented May 23, 2024

Describe the bug

On saving a new project, the Nav confirm Modal appears, asking if the user is they want to navigate away from the page without saving their data. This seems incorrect, as the the data IS saved. I'm also not sure if the user should be forced to leave the page after saving a project.

Furthermore, the user is sent into a non-existent, broken page in the calculator: http://localhost:3000/calculation/calculation/454

Steps to reproduce the issue

  1. Go to http://localhost:3000/calculation/1/0 "Create a New Project"
  2. Fill in whatever data is necessary to save a project in steps 1 - 3
  3. Save said project.

What's the expected result?

Project is saved.

What's the actual result?

Project is saved, but the nav confirm modal pops up, text off center, stating that the data is unsaved and asking if the user actually wishes to navigate away. Confirming sends the user to a broken page.

Additional details / screenshot

Screenshot 2024-05-22 at 5 00 05 PM
Screenshot 2024-05-22 at 5 00 01 PM

Update: Exclusive to newly created projects. Issue appears to be the ID of the newly saved project being placed into the URL and affecting things somehow.

Device configuration

@Jonathanko52 Jonathanko52 added bug Release Note: Shows as Error Correction priority: MUST HAVE labels May 23, 2024
@Parisajf Parisajf added this to the 10 - Launch milestone May 23, 2024
@Jonathanko52 Jonathanko52 self-assigned this May 29, 2024
@ExperimentsInHonesty ExperimentsInHonesty added the pbv: dev All engineering issues label Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Release Note: Shows as Error Correction level: medium p-feature: Save Button pbv: dev All engineering issues priority: MUST HAVE role: front-end Front End Developer size: 1pt Can be done in 4-6 hours
Projects
Status: On Dev - not yet pushed to Prod
Development

Successfully merging a pull request may close this issue.

3 participants