Skip to content

fix: patching blueprint with timeAfter=null not working#8517

Merged
narrowizard merged 1 commit intomainfrom
kw-8516-bp-timeafter-setnull
Jul 31, 2025
Merged

fix: patching blueprint with timeAfter=null not working#8517
narrowizard merged 1 commit intomainfrom
kw-8516-bp-timeafter-setnull

Conversation

@klesh
Copy link
Contributor

@klesh klesh commented Jul 31, 2025

Summary

Patching blueprint with timeAfter = null ends up with Zero-Value instead of nil

Does this close any open issues?

Closes #8516

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. component/framework This issue or PR relates to the framework needs-cherrypick-v1.0 pr-type/bug-fix This PR fixes a bug labels Jul 31, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 31, 2025
@narrowizard narrowizard merged commit f8246a7 into main Jul 31, 2025
15 checks passed
@narrowizard narrowizard deleted the kw-8516-bp-timeafter-setnull branch July 31, 2025 09:17
@github-actions
Copy link
Contributor

🤖 Target: #release-v1.0 cherry pick finished successfully 🎉!

@github-actions github-actions bot added the bot/auto-cherry-pick-completed auto cherry pick completed label Jul 31, 2025
narrowizard pushed a commit that referenced this pull request Jul 31, 2025
)

Co-authored-by: Klesh Wong <zhenmian.huang@merico.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot/auto-cherry-pick-completed auto cherry pick completed component/framework This issue or PR relates to the framework lgtm This PR has been approved by a maintainer needs-cherrypick-v1.0 pr-type/bug-fix This PR fixes a bug size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][core] PATCH API: Setting timeAfter to null results in 0001-01-01T00:00:00Z instead of no time limit

2 participants