-
Notifications
You must be signed in to change notification settings - Fork 42
Comparing changes
Open a pull request
base repository: temporalio/api-go
base: v1.43.0
head repository: temporalio/api-go
compare: v1.43.1
- 8 commits
- 44 files changed
- 7 contributors
Commits on Nov 27, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 0bc7dbc - Browse repository at this point
Copy the full SHA 0bc7dbcView commit details
Commits on Dec 4, 2024
-
Reserve RESET_REAPPLY_EXCLUDE_TYPE_CANCEL_REQUEST (#502)
_**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?** Reserve RESET_REAPPLY_EXCLUDE_TYPE_CANCEL_REQUEST option. <!-- Tell your future self why have you made these changes --> **Why?** Cancel request should be reapplied only in conflict resolution case. There is no option to exclude it for reset. <!-- 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** --------- Co-authored-by: Chad Retz <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for d08fda3 - Browse repository at this point
Copy the full SHA d08fda3View commit details
Commits on Dec 6, 2024
-
Add Failure to query result (#503)
**What changed?** Add a `Failure` field to: - QueryFailedFailure - WorkflowQueryResult - RespondQueryTaskCompletedRequest **Why?** Support encoding failure attributes for E2E encryption of messages and stack traces. **Server PR** Coming soon but I've already validated this change locally.
Configuration menu - View commit details
-
Copy full SHA for 4e70e5a - Browse repository at this point
Copy the full SHA 4e70e5aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8a5e148 - Browse repository at this point
Copy the full SHA 8a5e148View commit details -
Configuration menu - View commit details
-
Copy full SHA for b74f78e - Browse repository at this point
Copy the full SHA b74f78eView commit details
Commits on Dec 18, 2024
-
Add workflow expiration and run timestamps to execution info (#505)
_**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 new structure -WorkflowExecutionExtendedInfo That structure has few fields: * execution timeout time * run timeout time * last reset time * cancel requested (the last one from temporalio/api#339) <!-- Tell your future self why have you made these changes --> **Why?** Per customer request. Execution timeout timestamp may change after workflow reset, and in this case customers have no idea when it will fire. WorkflowExecutionInfo is reused in other server calls, like ListWorkflow, etc. Because of that we either add any new field to ES, or it will be nil which is bad user experience. <!-- 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** N/A
Configuration menu - View commit details
-
Copy full SHA for 91abcc4 - Browse repository at this point
Copy the full SHA 91abcc4View commit details
Commits on Dec 19, 2024
-
Clarify that Nexus Failure details must be JSON serializable (#506)
Co-authored-by: Spencer Judge <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 461db5a - Browse repository at this point
Copy the full SHA 461db5aView commit details
Commits on Jan 9, 2025
-
[Requires repository secret addition] Add Publish to Buf Github Actio…
…n (#508) _**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?** Adds a GitHub Actions workflow to automate the process of pushing protocol buffers to the Buf registry on every push to the main branch. We need to set the `BUF_TOKEN` secret before merging this PR. <!-- Tell your future self why have you made these changes --> **Why?** We want to be able to depend on server protos from other repositories (e.g. api-cloud) using the buf package manager. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** None. <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** NA
Configuration menu - View commit details
-
Copy full SHA for 915b673 - Browse repository at this point
Copy the full SHA 915b673View 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.43.0...v1.43.1