Skip to content

fix: document SvelteKit redirect URL and password reset email customization limit (gap report)#706

Open
tamalchowdhury wants to merge 2 commits intomainfrom
tamal/update/sveltekit-redirect-uri-and-password-reset-email-limits
Open

fix: document SvelteKit redirect URL and password reset email customization limit (gap report)#706
tamalchowdhury wants to merge 2 commits intomainfrom
tamal/update/sveltekit-redirect-uri-and-password-reset-email-limits

Conversation

@tamalchowdhury
Copy link
Copy Markdown
Collaborator

@tamalchowdhury tamalchowdhury commented Apr 7, 2026

This PR documents two limitations in Kinde:

SvelteKit uses one static KINDE_REDIRECT_URL (no built-in dynamic/multi-tenant callbacks; links to wildcards and “without an SDK” for alternatives).

Password reset mail can’t be fully custom HTML -only what the auth email templates allow. It also tightens email-content headings/asides and bumps frontmatter (TOC depth, dates).

Summary by CodeRabbit

  • Documentation
    • Clarified email customization docs: cleaned up headings, added an Aside for verification-note, and added an FAQ explaining password-reset HTML limits while confirming subject/body/disclaimers/placeholders remain customizable.
    • Improved SvelteKit SDK docs: better navigation (TOC up to h3) and new guidance on handling dynamic callback URLs and when to implement manual authorization flows.

@tamalchowdhury tamalchowdhury requested a review from a team as a code owner April 7, 2026 12:00
@github-actions github-actions bot added the sdk label Apr 7, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 9383f5ff-ef26-4b0f-87b0-9307f1151752

📥 Commits

Reviewing files that changed from the base of the PR and between d7e7ef1 and e658752.

📒 Files selected for processing (1)
  • src/content/docs/developer-tools/sdks/backend/sveltekit-sdk.mdx

Walkthrough

Documentation updates: email customization guide formatting and FAQ added; SvelteKit SDK guide added table-of-contents config and clarified static redirect behavior and guidance for dynamic callback needs. Metadata dates updated.

Changes

Cohort / File(s) Summary
Email Documentation Updates
src/content/docs/design/content-customization/email-content.mdx
Removed bold markup from several headings, converted an inline note into an <Aside> block, added an FAQ clarifying password reset emails cannot be fully custom HTML (subject/body/placeholders remain editable), and updated updated metadata to 2026-04-07.
SvelteKit SDK Documentation Updates
src/content/docs/developer-tools/sdks/backend/sveltekit-sdk.mdx
Added tableOfContents frontmatter (maxHeadingLevel: 3), added a section explaining the SDK uses a single static KINDE_REDIRECT_URL (no per-request dynamic redirect URIs) and advising manual auth-code flow for dynamic callbacks, and updated updated metadata to 2026-04-08.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through lines and smoothed the view,

Dropped the bold, and added something new,
TOC and FAQs in tidy rows,
Clearer paths where the rabbit goes,
A tiny edit, hopped—adios! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: documenting SvelteKit redirect URL limitations and password reset email customization limits, matching both file modifications in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tamal/update/sveltekit-redirect-uri-and-password-reset-email-limits

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.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 7, 2026

Deploying kinde-docs-preview with  Cloudflare Pages  Cloudflare Pages

Latest commit: e658752
Status: ✅  Deploy successful!
Preview URL: https://ff89487d.kinde-docs-preview.pages.dev
Branch Preview URL: https://tamal-update-sveltekit-redir.kinde-docs-preview.pages.dev

View logs


Kinde comes with a production environment, but you can set up other environments if you want to. Each environment has a unique subdomain so be sure to use the correct one in the **Configure your app section** which follows.

### Using dynamic callback URLs
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

3. In the subject line field, include the `${code}` placeholder. This is where the code will appear. E.g. Enter `${code}` for access.
4. Select **Save**.

## Can password reset emails use fully custom HTML?
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

@onderay onderay left a comment

Choose a reason for hiding this comment

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

A few comments

@tamalchowdhury tamalchowdhury changed the title ⚠️ fix: document SvelteKit redirect URL and password reset email customization limit (gap report) fix: document SvelteKit redirect URL and password reset email customization limit (gap report) Apr 8, 2026
@tamalchowdhury tamalchowdhury requested a review from onderay April 8, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants