Skip to content

fix PolicyResource: null date should not default to valid date#1215

Merged
deer-wmde merged 9 commits into
mainfrom
api_T432714-2
Jul 24, 2026
Merged

fix PolicyResource: null date should not default to valid date#1215
deer-wmde merged 9 commits into
mainfrom
api_T432714-2

Conversation

@deer-wmde

Copy link
Copy Markdown
Contributor

@deer-wmde
deer-wmde marked this pull request as ready for review July 24, 2026 12:13
Comment thread app/Http/Resources/PolicyResource.php Outdated
Comment thread tests/Resources/PolicyResourceTest.php Outdated
Comment thread tests/Resources/PolicyResourceTest.php
Comment thread tests/Resources/PolicyResourceTest.php Outdated
Comment thread tests/Resources/PolicyResourceTest.php
deer-wmde and others added 2 commits July 24, 2026 15:19
Co-authored-by: Dat WMDE <dat.nguyen@wikimedia.de>
Comment thread app/Http/Resources/PolicyResource.php Outdated

@dati18 dati18 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.

awesome work and collab. Approved!

@deer-wmde
deer-wmde merged commit 7f28ac7 into main Jul 24, 2026
4 checks passed
@deer-wmde
deer-wmde deleted the api_T432714-2 branch July 24, 2026 13:34
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