-
Notifications
You must be signed in to change notification settings - Fork 8
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
IS-2621: Add stansdato datepicker #1478
IS-2621: Add stansdato datepicker #1478
Conversation
c12b448
to
078e266
Compare
ae1d194
to
6038a16
Compare
const { datepickerProps, inputProps } = useDatepicker({ | ||
fromDate: varselSvarfrist, | ||
onDateChange: (date: Date | undefined) => { | ||
if (!!date) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trenger man håndtere setting og clearing av errors her?
Ser ikke ut som vi gjør det i AvslagDatePicker
og VedtakFraDato
hvor vi også har en påkrevd dato. Hadde vært nice å slippe kode for det her også, kanskje det er lettere om man skiller det ut som egen komponent? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Er litt redd for controlled components ettersom jeg ikke helt forstår hvordan det fungerer, men var ikke så mye greier så måtte til ser jeg. Endrer 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Synes det blir litt vel magisk med FormProvider
, men fant ikke en alternativ måte, så da blir det det 🥲
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blir bra det 👍
6038a16
to
29ae1df
Compare
Hva har blitt lagt til✨🌈
Legger til felt for stansdato på innstillingen om stans som går til NAY.
Se tilhørende PR i ismanglendemedvirkning
Screenshots 📸✨
Screen.Recording.2024-10-03.at.13.24.15.mov