Skip to content

docs: add go docs for payload storage#4424

Open
lennessyy wants to merge 12 commits intomainfrom
large-payload-go
Open

docs: add go docs for payload storage#4424
lennessyy wants to merge 12 commits intomainfrom
large-payload-go

Conversation

@lennessyy
Copy link
Copy Markdown
Contributor

@lennessyy lennessyy commented Apr 9, 2026

What does this PR do?

  • Add go docs for payload storage

Notes to reviewers

┆Attachments: EDU-6196 docs: add go docs for payload storage

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview, Comment Apr 10, 2026 5:03pm

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

📖 Docs PR preview links

@lennessyy lennessyy marked this pull request as ready for review April 9, 2026 20:15
@lennessyy lennessyy requested a review from a team as a code owner April 9, 2026 20:15
Copy link
Copy Markdown
Contributor

@flippedcoder flippedcoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of small things to consider!

layers that handle different concerns:

<CaptionedImage
src="/diagrams/data-converter-flow-with-external-storage.svg"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The arrows in and out of the container looked a little weird to me in dark mode. It could be fine, just worth another look.
Image

@@ -0,0 +1,315 @@
---
id: large-payload-storage
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could the id be changed to external-storage to match the page title? Maybe external-storage-aws if the other cloud platforms are coming later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants