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

feat: add cases of facility to frontend #252

Merged
merged 5 commits into from
Jul 12, 2024
Merged

feat: add cases of facility to frontend #252

merged 5 commits into from
Jul 12, 2024

Conversation

IngoSternberg
Copy link
Contributor

I also changed the routing of the goBack Button to use location.back(). I first thought just cutting the current url until the / would be the best but location.back() is actually better.

It goes back to the last page so now if you click on view 3rd case and go back you are redirected back to the details page where you came from

Signed-off-by: Ingo Sternberg [email protected]
#250

@IngoSternberg IngoSternberg linked an issue Jul 11, 2024 that may be closed by this pull request
4 tasks
IngoSternberg and others added 4 commits July 11, 2024 22:41
Signed-off-by: Ingo Sternberg <[email protected]>
#250
Co-authored-by: Ingo Sternberg <[email protected]>
Signed-off-by: Alexander Lorenz <[email protected]>
Copy link
Contributor

@HaruspexSan HaruspexSan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good - besides the two comments on the files, why is it called: "facility.cases.length" and not smth. like facility.cases.count as it is not the length of a case but i assume the number of cases for a given facility. This is nitpicking, i know.

@IngoSternberg
Copy link
Contributor Author

IngoSternberg commented Jul 12, 2024

It is .length because cases is an array and to get the length of it you do .length

Copy link
Contributor

@HaruspexSan HaruspexSan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good - tank you for the quick answer

@IngoSternberg IngoSternberg merged commit 5e86870 into develop Jul 12, 2024
4 checks passed
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.

Add cases of facility to Frontend
3 participants