Skip to content

[Workflows] Update rollback changelog#31264

Open
mia303 wants to merge 2 commits into
productionfrom
workflows/update-rollback-changelog
Open

[Workflows] Update rollback changelog#31264
mia303 wants to merge 2 commits into
productionfrom
workflows/update-rollback-changelog

Conversation

@mia303

@mia303 mia303 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@mia303 mia303 requested review from a team as code owners June 5, 2026 20:31
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Review

⏸️ Automatic reviews for this PR are paused.

This PR has already received 2 automatic reviews. To run another review, a codeowner can comment /review or /full-review.

Tip: Keep PRs in draft mode until they are ready for review — the bot skips draft PRs automatically.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/changelog/ @cloudflare/pm-changelogs, @cloudflare/product-owners

@ask-bonk ask-bonk Bot added the triage Waiting for a docs team review. label Jun 5, 2026

@ask-bonk ask-bonk Bot left a comment

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.

Updates the Workflows saga rollback changelog with a clearer, more concrete banking transfer example. One link fix and one content gap to address.

Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx Outdated
Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx
@ask-bonk

ask-bonk Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Review complete

Verdict: 1 fix needed, 1 content gap to consider. Posted 2 inline comments.

What changed: Updates the existing Workflows saga rollback changelog with a clearer title ("Workflows now supports saga-style rollbacks"), rewrites the opening paragraphs for readability, and replaces the generic provisionResource example with a concrete banking transfer scenario (debit/credit across accounts).

Issues:

  • MEDIUM — Missing trailing slash on internal link at line 11: [Workflows](/workflows) should be [Workflows](/workflows/). (Posted inline suggestion.)
  • MEDIUM — Content disconnect at line 58: the prose says rollback handlers support retry and timeout configuration, but the code example removed rollbackConfig that was present in the original. Either add it back to the example or drop the sentence. (Posted inline comment.)

Labels applied: triage

No build-breaking issues (frontmatter is valid, MDX characters are properly escaped, component imports are present).

github run

@mia303 mia303 force-pushed the workflows/update-rollback-changelog branch from 0e17c84 to 19547ea Compare June 5, 2026 20:45
Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx Outdated
@mia303 mia303 force-pushed the workflows/update-rollback-changelog branch 2 times, most recently from e6e1730 to 388ccab Compare June 5, 2026 21:05
@mia303 mia303 requested a review from irvinebroque June 6, 2026 13:07
Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx Outdated
Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx Outdated
Comment thread src/content/changelog/workflows/2026-06-05-saga-rollbacks.mdx Outdated
@github-actions

Copy link
Copy Markdown
Contributor

CI run failed: build logs

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

Labels

product:changelog size/s triage Waiting for a docs team review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants