Skip to content

Adding .NET SDK documentation for history propagation#5174

Open
WhitWaldo wants to merge 4 commits into
dapr:v1.18from
WhitWaldo:history-propagation
Open

Adding .NET SDK documentation for history propagation#5174
WhitWaldo wants to merge 4 commits into
dapr:v1.18from
WhitWaldo:history-propagation

Conversation

@WhitWaldo
Copy link
Copy Markdown
Contributor

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within tabpane
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have tabpane

In addition, please fill out the following to help reviewers understand this pull request:

Description

Added .NET SDK documentation for workflow history propagation functionality

Issue reference

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
@WhitWaldo WhitWaldo requested review from a team as code owners May 19, 2026 17:18
@github-actions github-actions Bot added the automerge: dotnet Auto-merged language-specific SDK docs label May 19, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Auto-approval: dotnet SDK docs

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Auto-approval: dotnet SDK docs

Copy link
Copy Markdown
Contributor

@nelson-parente nelson-parente left a comment

Choose a reason for hiding this comment

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

Pre-review for @marcduiker handoff: LGTM. New .NET SDK page for workflow history propagation. Content is well-structured: scopes table, propagating to child workflows, propagating to activities, reading history, and the PropagatedHistory type. Code examples use correct API surface (WithHistoryPropagation, HistoryPropagationScope, GetPropagatedHistory()). Version note (Dapr .NET SDK v1.18 + runtime v1.18) is present. Hugo front-matter looks good. Safe to merge after CI.

@WhitWaldo
Copy link
Copy Markdown
Contributor Author

@marcduiker Going to mark this as a do-not-merge momentarily as it needs updating to reflect another PR @nelson-parente opened that addresses some shortcomings in the first pass of this implementation.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Auto-approval: dotnet SDK docs

@msfussell msfussell added this to the v1.18 milestone May 24, 2026
@msfussell msfussell requested a review from cicoyle May 27, 2026 03:59
Copy link
Copy Markdown
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Auto-approval: dotnet SDK docs

@cicoyle
Copy link
Copy Markdown
Contributor

cicoyle commented May 27, 2026

lgtm but CI is 🔴 Mind taking a look @WhitWaldo ?

@WhitWaldo
Copy link
Copy Markdown
Contributor Author

@cicoyle Nelson is still wrapping up the revised implementation, so these documents are probably out of date. After his PR is merged, I'll update these docs and figure out why CI is unhappy.

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

Labels

automerge: dotnet Auto-merged language-specific SDK docs do-not-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants