-
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.47.0
head repository: temporalio/api-go
compare: v1.48.0
- 7 commits
- 12 files changed
- 5 contributors
Commits on Apr 9, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 8b109c8 - Browse repository at this point
Copy the full SHA 8b109c8View commit details -
Support visiting repeated Payload (#216)
* Support repeated Payload, add godoc comment regarding not supporting Payload directly * Forgot to commit a change, remove slices import
Configuration menu - View commit details
-
Copy full SHA for a4daf74 - Browse repository at this point
Copy the full SHA a4daf74View commit details
Commits on Apr 10, 2025
-
Add reason to the PauseActivityRequest (#568)
_**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 "reason" to PauseActivityRequest <!-- Tell your future self why have you made these changes --> **Why?** Product request. Unlike other activity operation, pause is reversible. For manually paused activities operator may need to know the reason for pausing. <!-- 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**
Configuration menu - View commit details
-
Copy full SHA for 3a3166a - Browse repository at this point
Copy the full SHA 3a3166aView commit details
Commits on Apr 16, 2025
-
Add PauseInfo to the pending activity info (#569)
<!-- Describe what has changed in this PR --> **What changed?** Add "PauseInfo" message to the PendincgActivityInfo. Message contain information about who, why and when paused the activity. Note: if activity is paused by rule id, users will need to call "DescribeWorkflowRule" to get the details for that rule. <!-- Tell your future self why have you made these changes --> **Why?** Product request, for audit purpose. Note: we don't add this information to the history. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** No
Configuration menu - View commit details
-
Copy full SHA for 8d5dc33 - Browse repository at this point
Copy the full SHA 8d5dc33View commit details -
Clarify docs of root_workflow_execution in WorkflowExecutionStartedEv…
…entAttributes (#573) _**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?** Clarify docs of `root_workflow_execution` in `WorkflowExecutionStartedEventAttributes` <!-- Tell your future self why have you made these changes --> **Why?** <!-- 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 2f6214c - Browse repository at this point
Copy the full SHA 2f6214cView commit details -
Add ScheduledEventID to NexusOperationCancelRequest[Completed|Failed]…
… (#572) **What changed?** Added ScheduledEventID of the Nexus operation to the new NexusOperationCancelRequestCompleted and NexusOperationCancelRequestFailed events **Why?** For efficient lookups. Go SDK stores cancellation state machines keyed on scheduled event ID. **Server PR** temporalio/temporal#7612
Configuration menu - View commit details
-
Copy full SHA for b89b9b2 - Browse repository at this point
Copy the full SHA b89b9b2View commit details -
Add ApplicationErrorCategory to ApplicationFailureInfo (#570)
<!-- Describe what has changed in this PR --> **What changed?** Added an ENUM `ApplicationErrorCategory`, used in `ApplicationFailureInfo` proto. <!-- Tell your future self why have you made these changes --> **Why?** Give users a way to categorize application failures. Error categories map to different observability behaviours SDK-side (i.e. logging/metrics), part of the work for benign exceptions. Categories could also be used with rules in the future. <!-- 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** Don't believe this breaks server or requires any changes to server
Configuration menu - View commit details
-
Copy full SHA for 71e208d - Browse repository at this point
Copy the full SHA 71e208dView 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.47.0...v1.48.0