-
Notifications
You must be signed in to change notification settings - Fork 37
Comparing changes
Open a pull request
base repository: temporalio/api-go
base: v1.48.0
head repository: temporalio/api-go
compare: v1.49.0
- 7 commits
- 37 files changed
- 4 contributors
Commits on Apr 23, 2025
-
Add request ID reference in links (#563)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** Add request ID reference to `Link`. <!-- Tell your future self why have you made these changes --> **Why?** Request ID can be used to associate links to history event ID. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR**
Configuration menu - View commit details
-
Copy full SHA for eb8d7ea - Browse repository at this point
Copy the full SHA eb8d7eaView commit details
Commits on Apr 24, 2025
-
Re-enable test visiting payloads in error details (#219)
* Enable section of test that is now supported * Forgot a change
Configuration menu - View commit details
-
Copy full SHA for 7ad90b7 - Browse repository at this point
Copy the full SHA 7ad90b7View commit details
Commits on Apr 26, 2025
-
Add indentity/description to the rule, propagate then in PauseInfo (#…
…577) <!-- Describe what has changed in this PR --> **What changed?** 1. Add identity/description to the rules. 2. Add new message with "rule_id, identity, reason" to the PauseInfo <!-- Tell your future self why have you made these changes --> **Why?** Product request - identity/reason should be always populated, for both manual/rule. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** Not really. But old "rule_id" in oneof is effectively deprecated.
Configuration menu - View commit details
-
Copy full SHA for d6c89de - Browse repository at this point
Copy the full SHA d6c89deView commit details
Commits on Apr 29, 2025
-
💥[breaking] Remove cloud files (#218)
* Remove cloud files * Remove cloud from Makefile and proxygenerator
Configuration menu - View commit details
-
Copy full SHA for b910ee7 - Browse repository at this point
Copy the full SHA b910ee7View commit details
Commits on May 1, 2025
-
Support parsing error details that aren't anypb.Any type (#214)
* Support parsing details that aren't anypb.Any type * PR feedback, simplify code * Visit response on non-status errors * PR feedback * move inbound check * Stop wrapping status errors in any * refactor error handling and avoid inline type definitions * refactor error handling and avoid inline type definitions
Configuration menu - View commit details
-
Copy full SHA for c479868 - Browse repository at this point
Copy the full SHA c479868View commit details -
Add
status
to StartWorkflowExecutionResponse (#580)_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** Added field `WorkflowExecutionStatus status` to `StartWorkflowExecutionResponse`. <!-- Tell your future self why have you made these changes --> **Why?** To align Update-with-Start's behavior with a regular Update. A regular Update returns its outcome from a closed Workflow, to make Update-with-Start do the same thing, it needs to returns a successful response in that case. But that would tell the user that a running Workflow exists. To help the user distinguish whether UwS operated on a closed or running workflow, this field is introduced. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** No. <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** temporalio/temporal#7656
Configuration menu - View commit details
-
Copy full SHA for 268e4b2 - Browse repository at this point
Copy the full SHA 268e4b2View commit details -
Add link field to Callback (#576)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** Add `links` field to `Callback`. <!-- Tell your future self why have you made these changes --> **Why?** Be able to explicitly associate a callback with links. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR**
Configuration menu - View commit details
-
Copy full SHA for 5045ddc - Browse repository at this point
Copy the full SHA 5045ddcView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v1.48.0...v1.49.0