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.

Sign up for GitHub

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

Jump to bottom

Feature/better trigger handling #3676

Merged
merged 4 commits into from
Nov 29, 2024
Merged

Feature/better trigger handling #3676

merged 4 commits into from
Nov 29, 2024

Conversation

stigus
Copy link
Contributor

@stigus stigus commented Nov 26, 2024

Endrer hvordan react-hook-forms håndterer trigger og fikset ugyldig date format

@stigus
- Bedre handling av react-hook-forms trigger sjekking
b7c3ef1 
#deploy-test-frontend
@stigus
Merge branch 'master' into feature/better-trigger-handling
8212263
@stigus
- Bedre handling av react-hook-forms trigger sjekking
0a685ea 
- Fikset date fra norsk bankkonto på ugyldig format, muligens andre steder
- Gir rød input boks dersom feltet ikke validerer
#deploy-test-frontend
@stigus stigus requested a review from a team as a code owner November 26, 2024 08:05
betsytraran
betsytraran approved these changes Nov 26, 2024
View reviewed changes
Copy link
Contributor

@betsytraran betsytraran left a comment

Choose a reason for hiding this comment

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

Ser fint ut, men jeg er litt usikker på hvordan den funker med de "manuelle" feilmeldingene våre 🤔 Bare testet et sted jeg vet om, på skriv inn orgnummer på inntektstub, og der forsvinner feilmeldingen når man prøver å gå videre 😬

@stigus
Copy link
Contributor Author

stigus commented Nov 26, 2024

Ser fint ut, men jeg er litt usikker på hvordan den funker med de "manuelle" feilmeldingene våre 🤔 Bare testet et sted jeg vet om, på skriv inn orgnummer på inntektstub, og der forsvinner feilmeldingen når man prøver å gå videre 😬

Ja jeg gjorde det bare for datepicker siden endringene var der, men kan vel fikse dette alle steder vi bruker setError når jeg først er i gang 🤪

@betsytraran
Copy link
Contributor

Ser fint ut, men jeg er litt usikker på hvordan den funker med de "manuelle" feilmeldingene våre 🤔 Bare testet et sted jeg vet om, på skriv inn orgnummer på inntektstub, og der forsvinner feilmeldingen når man prøver å gå videre 😬

Ja jeg gjorde det bare for datepicker siden endringene var der, men kan vel fikse dette alle steder vi bruker setError når jeg først er i gang 🤪

Aaah, skjønner, men nice nice nice! 🤩

@stigus
- Større trigger endringer, lagt til bedre støtte for manuelle feil s…
f84119d 
…att fra koden istedenfor yup-validering

- Noe mindre bugfixing av diverse elementer funnet under testing
#deploy-test-frontend
@stigus stigus merged commit 0840efb into master Nov 29, 2024
12 checks passed
@stigus stigus deleted the feature/better-trigger-handling branch November 29, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@betsytraran betsytraran betsytraran approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stigus @betsytraran