fix: no redirect on save #599
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hegeaal
approved these changes
Oct 13, 2023
jeffreiffers
requested changes
Oct 13, 2023
pooriamehregan
force-pushed
the
fix/por/no-redirect
branch
from
b091dee to
2e07107
Compare
NilsOveTen
reviewed
Oct 17, 2023
| @@ -196,6 +196,12 @@ export const FormConceptPure: FC<Props> = ({ | |||
| return () => {}; | |||
| }, [dirty, showUserPrompt, saveCalled, deleteCalled]); | |||
|
|
|||
| useEffect(() => { | |||
| if (saveCalled) { | |||
| window.location.reload(); | |||
There was a problem hiding this comment.
Hva skjer hvis du nå lagrer et helt nytt begrep?
There was a problem hiding this comment.
Tror lagringsfunksjonaliteten vil fungere som før, men i stedet for å bli redirectet til begrepskatalogen etter lagring, så refreshes siden på nytt for å hente sise oppdateringene som ble lagret etter click på lagre-knappen.
There was a problem hiding this comment.
joa, men vil den da ikke tro at du vil opprette enda et nytt begrep?
siden du ikke har id i uri-feltet
There was a problem hiding this comment.
Har testet nå. Den lander på samme uri hver gang. Men kan hende jeg misforstår deg
pooriamehregan
force-pushed
the
fix/por/no-redirect
branch
from
2e07107 to
3a6225a
Compare
pooriamehregan
force-pushed
the
fix/por/no-redirect
branch
from
3a6225a to
273dfb3
Compare
|
Denne kan lukkes! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes #585