Skip to content

chore: update urls - #33

Open
Olexandr88 wants to merge 1 commit into
zkemail:mainfrom
Olexandr88:patch-1
Open

chore: update urls#33
Olexandr88 wants to merge 1 commit into
zkemail:mainfrom
Olexandr88:patch-1

Conversation

@Olexandr88

@Olexandr88 Olexandr88 commented Dec 7, 2025

Copy link
Copy Markdown

Summary by CodeRabbit

Release Notes

  • Documentation
    • Updated ZK Email Registry URL references in the SDK documentation to direct to the new registry location.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Dec 7, 2025

Copy link
Copy Markdown

Walkthrough

Two hyperlink references in the ZK Email SDK documentation were updated from the legacy SDK URL to the new registry URL. The links in the "Getting started" and "Access the ZK Email Registry" sections now point to https://registry.zk.email/ instead of https://sdk.prove.email/.

Changes

Cohort / File(s) Summary
Documentation URL Updates
docs/zk-email-sdk/creating-a-new-pattern.md
Updated two hyperlink targets from https://sdk.prove.email/ to https://registry.zk.email/ in the Getting started section and Access the ZK Email Registry subsection

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify that the new registry URL is correct and functional
  • Confirm both link locations have been properly updated

Poem

🐰 A URL tweak, so simple and neat,
From old prove.email to registry's beat,
The docs now point true, to the right destination,
Like carrots rephrased for a fresh interpretation! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore: update urls' is generic and vague, using non-descriptive language that doesn't convey which URLs were updated or why. Consider a more specific title like 'chore: update ZK Email Registry URLs' to clarify which URLs are being updated and provide better context for reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dbd0360 and 1942c17.

📒 Files selected for processing (1)
  • docs/zk-email-sdk/creating-a-new-pattern.md (2 hunks)
🔇 Additional comments (3)
docs/zk-email-sdk/creating-a-new-pattern.md (3)

24-24: URL migration correctly applied to "Getting started" section.

The URL has been properly updated from the old registry location to https://registry.zk.email/, which is the new registry location, while the older version is being phased out.


42-42: URL migration correctly applied to "Access the ZK Email Registry" section.

The link in this section has been properly updated to the new official registry URL. Both hyperlink updates in the file now consistently reference the migrated registry location.


1-251: Verify no stale references to the old SDK URL remain in this file.

The provided code snippet (lines 1-251) shows no references to the old sdk.prove.email domain. All URLs reference the current registry.zk.email endpoint and legitimate documentation sources. However, direct repository access is required to definitively confirm the entire file contains no stale references.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant