zaakstatus datumstatusgezet fix incl. validation#116
Conversation
heuvea
added
Bug
There was a problem hiding this comment.
Pull request overview
This PR fixes the handling of datetime fields for zaakstatus, specifically addressing the datumStatusGezet field format and adding proper validation. The changes ensure datetime values are correctly formatted as UTC/ISO 8601 and validated to prevent future dates.
Key changes:
- Changed datetime formatting from local time to UTC with explicit 'Z' timezone indicator
- Enhanced the
NotInTheFuturevalidation to properly parse UTC timestamps with culture-invariant settings
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/OneGround.ZGW.Common/Helpers/ProfileHelper.cs | Updated SortableStringDateFromDate to format dates as UTC ISO 8601 with explicit format string |
| src/OneGround.ZGW.Common.Web/Validations/IRuleBuilderExtensions.cs | Enhanced NotInTheFuture validation to properly parse UTC datetime strings with culture-invariant parsing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
heuvea
changed the title
ragaumon
force-pushed
the
bugfix/FUND-2188-ZRC-get-status-does-not-return-Z-for-time
branch
2 times, most recently
from
4c19e2f to
4f01328
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ragaumon
force-pushed
the
bugfix/FUND-2188-ZRC-get-status-does-not-return-Z-for-time
branch
from
37ce57c to
4f01328
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ragaumon
force-pushed
the
bugfix/FUND-2188-ZRC-get-status-does-not-return-Z-for-time
branch
from
4f01328 to
c423668
Compare
ragaumon
deleted the
bugfix/FUND-2188-ZRC-get-status-does-not-return-Z-for-time
branch
Pull Request
Description
Type of Change
Related Issues
Testing
Checklist