Conversation
|
@yuandrew is attempting to deploy a commit to the Temporal Team on Vercel. A member of the Team first needs to authorize it. |
|
Reviewing, editing, and merging this today. |
|
Go SDK v1.40.0 was just released today, so the SDK should now support all of these code snippets for users to use themselves https://github.com/temporalio/sdk-go/releases/tag/v1.40.0. So good to merge whenever, from my side! Thanks @jsundai |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@prasek Could you take a quick look at this when you get a chance? Thanks! |
…tion into standalone-activities
|
@jsundai Phil is pretty backed up with other tasks. We should be good to merge here, we can always update the docs when Phil gets a chance to review later on |
|
This looks good to me. Since it's experimental, we'll create another follow up PR to change it to pre-release next week. |
|
|
||
| :::info PREREQUISITES | ||
|
|
||
| Standalone Activities require server-side support. If you are running a self-hosted Temporal Server, the following |
There was a problem hiding this comment.
Remove this section. This feature will only work on a special CLI release and in Temporal Cloud. In server 1.31.0 (the next OSS server release) all of this will be enabled by default.
| You can pass the Activity as either a function reference or a string Activity type name: | ||
|
|
||
| ```go | ||
| // Using a function reference (recommended — enables compile-time parameter validation) |
There was a problem hiding this comment.
That's false. It's runtime parameter validation (assuming that was implemented).
prasek
left a comment
prasek
left a comment
There was a problem hiding this comment.
overall looks good, added some suggestions
|
|
||
| ::: | ||
|
|
||
| :::info PREREQUISITES |
There was a problem hiding this comment.
We should have them run with dev server, similar to https://docs.temporal.io/develop/typescript/nexus#run-the-temporal-nexus-development-server
@dandavison @bergundy will there be a temporal server start-dev flag to enable Standalone Activities?
There was a problem hiding this comment.
No, there will not be a flag. The special release of the dev server will enable it by default.
| - [Get the result of a Standalone Activity](#get-activity-result) | ||
| - [Get a handle to an existing Standalone Activity](#get-activity-handle) | ||
|
|
||
| :::caution EXPERIMENTAL |
There was a problem hiding this comment.
This should say it's in Pre-release using the same form as the Nexus SDK feature guide
Like this
:::tip SUPPORT, STABILITY, and DEPENDENCY INFO
Temporal Go SDK support for Standalone Activities is at [Pre-release](https://docs.temporal.io/evaluate/development-production-features/release-stages#pre-release).
All APIs are experimental and may be subject to backwards-incompatible changes.
:::
|
|
||
| :::info PREREQUISITES | ||
|
|
||
| Standalone Activities require server-side support. If you are running a self-hosted Temporal Server, the following |
There was a problem hiding this comment.
These self-hosted options should be moved to the self-hosted guide, similar to https://docs.temporal.io/production-deployment/self-hosted-guide/nexus
Also we should stick with dev server (and Cloud) for these SDK dev guides.
There was a problem hiding this comment.
This isn't available and not relevant to document.
| ::: | ||
|
|
||
|
|
||
| ## Execute a Standalone Activity {#execute-activity} |
There was a problem hiding this comment.
ideally this quick start should allow users to cut/paste commands and have it working on their local machine with the least amount of work possible
so before this would add:
- note saying everything is based on an SDK sample (with link), so they can clone it
- commands (they can cut/paste) to clone, get dev server initialized, and the sample activity worker running, so it's ready to for a client to execute a Standalone Activity
- then after this code walkthrough, would provide a command (they can cut/paste) to execute the standalone activity
|
What about list and count for standalone activities? |
…tion into standalone-activities
|
@jsundai we decided this should wait until a special server release that will fully support standalone activities for this to go live, converting to a draft for now. Will re-publish when we're ready to merge this |
4 participants
What does this PR do?
Describes a new feature for standalone activities
Notes to reviewers