Skip to content

docs(materialize): remove author draft/TODO callouts from concept pages#568

Merged
Corey-T1000 merged 1 commit into
mainfrom
docs/remove-materialize-todo-callouts
Jul 9, 2026
Merged

docs(materialize): remove author draft/TODO callouts from concept pages#568
Corey-T1000 merged 1 commit into
mainfrom
docs/remove-materialize-todo-callouts

Conversation

@Corey-T1000

@Corey-T1000 Corey-T1000 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #562. The Materialize concept pages shipped with author-to-author
<Callout> notes that render as live info/warning boxes to readers. This removes
them from both pages:

app/materialize/concepts/permission-set-lifecycle/page.mdx

  • "Draft — content review needed" banner
  • TODO callout: DownloadPermissionSets reference page + worked-example expansion

app/materialize/concepts/snapshots/page.mdx

  • "Draft — content review needed" banner
  • TODO callout: snapshot retention-window confirmation

The underlying follow-up work (a DownloadPermissionSets reference page, a worked
backfill example, and confirming snapshot retention/rotation) is tracked separately —
this PR just stops the notes from rendering to readers. The nextra Callout import
stays; both pages still use it for genuine reader-facing callouts.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Preview deployment status for this pull request.

Name Status Preview Updated (UTC)
docs 🟢 Ready Visit Preview Jul 09, 2026 03:48pm

@authzed-catherine authzed-catherine 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.

I think there's one more to remove at the top of the snapshots page.

The permission-set-lifecycle and snapshots concept pages carried author-to-author
<Callout> notes that render as live info/warning boxes to readers:

- "Draft — content review needed" banners at the top of both pages
- TODO callouts (DownloadPermissionSets reference-page, worked-example expansion,
  and snapshot retention-window confirmation — a couple addressed to Sam)

Remove them. The real follow-up work they flagged is tracked separately; this just
stops the notes from shipping to readers. The nextra Callout import stays — both
pages still use it for genuine reader-facing callouts.
@Corey-T1000
Corey-T1000 force-pushed the docs/remove-materialize-todo-callouts branch from 7a6b5de to a20212e Compare July 9, 2026 15:45
@Corey-T1000 Corey-T1000 changed the title docs(materialize): remove author TODO callouts from concept pages docs(materialize): remove author draft/TODO callouts from concept pages Jul 9, 2026
@Corey-T1000
Corey-T1000 merged commit 1075192 into main Jul 9, 2026
12 of 13 checks passed
@Corey-T1000
Corey-T1000 deleted the docs/remove-materialize-todo-callouts branch July 9, 2026 15:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants