refactor(go): updated /develop/go to restructure content#4250
Open
flippedcoder wants to merge 8 commits intomm/restructure-sdk-contentfrom
Open
refactor(go): updated /develop/go to restructure content#4250flippedcoder wants to merge 8 commits intomm/restructure-sdk-contentfrom
flippedcoder wants to merge 8 commits intomm/restructure-sdk-contentfrom
Conversation
Contributor
📖 Docs PR preview links
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
* fix(task queues): updated pricing info for fairness * chore(task queues): clean up based on feedback
* Clarify message idempotency section * v2 * v3 * Apply suggestion from @flippedcoder * Apply suggestion from @flippedcoder --------- Co-authored-by: Milecia McG <47196133+flippedcoder@users.noreply.github.com>
…nd workflow IDs (#4301) * Strengthen Search Attribute guidance on sensitive data and PII Elevate the existing note about unencrypted Search Attributes to a caution admonition with expanded guidance covering both attribute names and values. Add reasoning about compliance risk (GDPR, HIPAA, SOC 2) and recommend using opaque identifiers instead. Cross-reference the warning from the Default Search Attributes section. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * drop line about hashing * drop heavy handed redirect * Add sensitive data warning to Workflow Id documentation Workflow Ids and other user-defined identifiers (Workflow Type, Task Queue, Activity, Signal/Query/Update names) are stored in plain text and bypass the Payload Codec. Add a caution admonition warning against using PII directly as a Workflow Id, with compliance reasoning (GDPR, HIPAA, SOC 2). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: add admonition in custom search attribute section in visibility * docs: use more specific anchors --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Lenny Chen <lenny.chen@temporal.io>
flippedcoder
force-pushed
the
mm/go-core-apps-decouple
branch
from
d1c0fb5 to
37e1df0
Compare
flippedcoder
force-pushed
the
mm/go-core-apps-decouple
branch
from
9c2efb4 to
d1ea140
Compare
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This refactors the
/develop/gosection of the docs to change how we organized content.activities,client,nexus,workflows,workers, andbest practicessub-sectionscore-applicationspage into multiple pages that are distributed to the above sub-sectionsNotes to reviewers
┆Attachments: EDU-5961 refactor(go): updated /develop/go to restructure content