Skip to content

[fix] Restricts update date validation for non-maintenance events#204

Merged
otc-zuul[bot] merged 3 commits into
mainfrom
i185
Jul 22, 2025
Merged

[fix] Restricts update date validation for non-maintenance events#204
otc-zuul[bot] merged 3 commits into
mainfrom
i185

Conversation

@Aloento
Copy link
Copy Markdown
Member

@Aloento Aloento commented Jul 21, 2025

Updates the update date validation logic to only enforce that the update date cannot be earlier than the start date for non-maintenance event types.

This change allows maintenance events to bypass this restriction, accommodating different scheduling needs.

Updates the update date validation logic to only enforce that the update date cannot be earlier than the start date for non-maintenance event types.

This change allows maintenance events to bypass this restriction, accommodating different scheduling needs.
@Aloento Aloento self-assigned this Jul 21, 2025
@Aloento Aloento linked an issue Jul 21, 2025 that may be closed by this pull request
@sgmv
Copy link
Copy Markdown
Contributor

sgmv commented Jul 21, 2025

You should exhibit this behaviour not only for maintenance, but also for the info event.

Copy link
Copy Markdown
Contributor

@sgmv sgmv left a comment

Choose a reason for hiding this comment

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

Please add the same for the info event type.

Updates the date validation logic to ensure that the update date cannot be earlier than the start date for non-incident event types.

This change improves the accuracy of the event management process by preventing invalid date entries.
@Aloento Aloento requested a review from sgmv July 21, 2025 14:30
Adjusts the condition for validating the update date to ensure it only restricts incidents when the update date is earlier than the start date.

This change enhances the user experience by preventing incorrect date entries for incident events.
@Aloento Aloento added the gate Merge PR label Jul 22, 2025
@otc-zuul otc-zuul Bot merged commit 7288694 into main Jul 22, 2025
7 checks passed
@otc-zuul otc-zuul Bot deleted the i185 branch July 22, 2025 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate Merge PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't update the maintenance, update date is earlie

2 participants