Skip to content

Fix decoding of non-nullable interval fields when null is gotten#570

Merged
roxblnfk merged 3 commits into
masterfrom
fix
Feb 19, 2025
Merged

Fix decoding of non-nullable interval fields when null is gotten#570
roxblnfk merged 3 commits into
masterfrom
fix

Conversation

@roxblnfk

@roxblnfk roxblnfk commented Feb 19, 2025

Copy link
Copy Markdown
Collaborator

What was changed

After a detailed review of the codebase and running various tests, it turned out that Issue #481 was not confirmed. Workflow Types do not leak beyond their TaskQueue. The reason for the failing tests was in the test settings in Harness.

Another conversion of null values to an empty interval (valid only for Cloud environments) was found and fixed.

Checklist

  1. Closes [Bug] Unrecognized date time interval format on unmarshalling #555
  2. Closes The SDK does not use TaskQueue when searching for the Workflow to be started. #481
  3. How was this tested:
    • added autotests
    • tested on Cloud in Harness

@vercel

vercel Bot commented Feb 19, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
php ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2025 10:29am

Disable MissingOverrideAttribute and ClassMustBeFinal
@roxblnfk roxblnfk added this to the 2.13.2 milestone Feb 19, 2025
@roxblnfk roxblnfk added the Bug Something isn't working label Feb 19, 2025
@roxblnfk
roxblnfk merged commit 347e344 into master Feb 19, 2025
@roxblnfk
roxblnfk deleted the fix branch February 19, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Unrecognized date time interval format on unmarshalling The SDK does not use TaskQueue when searching for the Workflow to be started.

2 participants