Skip to content

🔒 Refresh Privacy on Stellar guide - #2673

Draft
JFWooten4 wants to merge 6 commits into
stellar:mainfrom
JFWooten4:docs/use-platform-api-doc-cards
Draft

🔒 Refresh Privacy on Stellar guide#2673
JFWooten4 wants to merge 6 commits into
stellar:mainfrom
JFWooten4:docs/use-platform-api-doc-cards

Conversation

@JFWooten4

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 22, 2026 09:07
Co-authored-by: Codex <noreply@openai.com>

Copilot AI 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.

Pull request overview

Refreshes the Stellar privacy guide for clarity and consistency.

Changes:

  • Improves wording, punctuation, and terminology.
  • Standardizes links and resource labels.
  • Reformats ZK primitives as a list.

Protocol 26 is cited but its CAP-80 host functions are omitted.

Recommendation: NEEDS-CHANGES — document Protocol 26’s CAP-80 additions.

Comment thread docs/build/apps/privacy.mdx
Co-authored-by: Codex <noreply@openai.com>

@ElliotFriend ElliotFriend 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.

Thanks, John! The CAP-80 addition is the real win here. Good catch that it was missing, and the bulleted list reads better. Dropping the duplicate H1 is right too.

Could you trim it to that? The rest is mostly cosmetic: em-dash spacing, a couple trailing-slash removals, and the date reformat (which actually goes against the Month DD, YYYY style used elsewhere on the page). I'd rather keep those out so the diff stays focused on the CAP-80 content. Keep the genuine fixes (the Forfor typo, UltraHonk casing, grammar), just lose the style churn. Then I'll be happy to get it merged.

@JFWooten4
JFWooten4 marked this pull request as draft July 28, 2026 20:01
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.

3 participants