Skip to content

fix(ARC-3691): fix date pickers in create maintenance alert blade#347

Open
bertyhell wants to merge 1 commit into
mainfrom
bugfix/ARC-3691-date-pickers
Open

fix(ARC-3691): fix date pickers in create maintenance alert blade#347
bertyhell wants to merge 1 commit into
mainfrom
bugfix/ARC-3691-date-pickers

Conversation

@bertyhell

@bertyhell bertyhell commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

@reunefe reunefe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is het niet beter om een parser voor de datum te schrijven?

@bertyhell

Copy link
Copy Markdown
Contributor Author

Is het niet beter om een parser voor de datum te schrijven?

hoe bedoel je precies?
die parseAsIsoWithoutTimezone(maintenanceAlert.fromDate).toISOString() is vooral om die tijdzone te negeren bij de datum selectie, anders wordt de dag voordien geselecteerd omdat er 2 uur verschil op zit

@bertyhell bertyhell requested a review from reunefe June 22, 2026 16:56
@reunefe

reunefe commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Is het niet beter om een parser voor de datum te schrijven?

hoe bedoel je precies? die parseAsIsoWithoutTimezone(maintenanceAlert.fromDate).toISOString() is vooral om die tijdzone te negeren bij de datum selectie, anders wordt de dag voordien geselecteerd omdat er 2 uur verschil op zit

Dat snap ik. Maar je bent nu 4x dezelfde logica of bijna dezelfde logica aan het doen. Lijkt me handiger dat dat in 1 functie zit?

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.

2 participants