AC-493 feature: create an edit-test-center on front end with prepopul…

[MattMoran12]

…ated for an edit request

Checklist for Pull Requests

🚨Please review the guidelines for contributing to this repository.

• [ ] Make sure your branch matches the naming conventions. (Something like: AC-152). Don't request your master!
• [ ] Your branch should only contain changes relevant to the ticket it is named after.
• [ ] Make sure you are making a pull request against the dev branch (left side). Also you should start your branch off our dev.
• [ ] Check your code additions will fail neither code linting checks nor e2e test (this will be tested automatically when you open your PR).

Description

• created "edit-test-center" page to replace old test edit page
• page will prepopulate all relevant data for user to update or add, then submit
• url for page is path="/edit-test-center/:id"
• accessing page with invalid id will redirect to prior page or map, with message appearing in the snackbar
• link to ticket: https://trello.com/c/2gMHlJm3/61-good-citizen-edit-location-ac-493

[crimsonsunset]

sorry, can you explain whats supposed to happen when user hits submit? id imagine theres some kind of QA process on our side? like random logged out users shouldnt be able to just edit our locations right?

right now, when i submit a change and then refresh the page, i dont see any of the changes i just put in.

[MattMoran12]

sorry, can you explain whats supposed to happen when user hits submit? id imagine theres some kind of QA process on our side? like random logged out users shouldnt be able to just edit our locations right?

right now, when i submit a change and then refresh the page, i dont see any of the changes i just put in.

Yes so this page should request an edit, not make the edit. It calls the requestChangeByCitizen endpoint which should add this request to a queue which will be looked at before the change is actually made to the test center.

[MattMoran12]

By putting the duplicate code into a component, I was able to cut the duplicate code down to 19.1%. Most of the duplicate code is due to the functions being passed in to the component. These functions have small difference or might be handled different in the future which is why I'm unsure how to cut down even further.

[sonarcloud[bot]]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
19.1% Duplication