Skip to content

feat: add "Why do I need to login?" link to onboarding sign-up#4211

Open
devin-ai-integration[bot] wants to merge 7 commits intomainfrom
devin/1771941316-why-login-faq-link
Open

feat: add "Why do I need to login?" link to onboarding sign-up#4211
devin-ai-integration[bot] wants to merge 7 commits intomainfrom
devin/1771941316-why-login-faq-link

Conversation

@devin-ai-integration
Copy link
Contributor

Summary

Adds a "Why do I need to login?" hyperlink to the sign-up/login screens (both web auth page and desktop onboarding) that points to a new FAQ entry explaining why account creation exists in a local-first app.

Changes:

  • FAQ (apps/web/content/docs/faq/0.general.mdx): New "Why do I need to create an account?" section explaining that accounts unlock cloud features while local data is unaffected
  • Web auth (apps/web/src/routes/auth.tsx): WhyLoginLink component rendered between the sign-in buttons and legal text on the main auth view
  • Desktop onboarding (apps/desktop/src/components/onboarding/account/before-login.tsx): Same link added below the sign-in button

Review & Testing Checklist for Human

  • Verify the anchor link works: The link targets https://char.com/docs/faq/general#why-do-i-need-to-create-an-account — confirm the MDX heading generates this exact anchor slug on the live docs site. If the docs framework slugifies differently, the link will land on the page but not scroll to the section.
  • FAQ says account creation is "optional" — verify this is accurate. The onboarding config (config.tsx) includes "login" as a required step in both macOS and other platform flows. If users can't skip login during onboarding, "optional" may be misleading.
  • Hardcoded https://char.com URL — the link won't resolve correctly on preview/staging deploys (e.g., Netlify deploy previews). Consider whether a relative path (/docs/faq/general#...) would be better for the web auth page. The desktop app legitimately needs the absolute URL.
  • Visual check: Confirm the link looks right in both the web auth page (between sign-in buttons and legal text) and the desktop onboarding (below the sign-in button). The link only appears on the main OAuth view, not the email sign-in sub-view.

Notes

devin-ai-integration bot and others added 7 commits February 24, 2026 08:37
- Update README 'Truly Local' section to explain account creation during onboarding and local data storage
- Add Local-First Architecture and Account Deletion sections to Terms of Service
- Add Local-First Design section to Privacy Policy with updated section numbering
- Add 'Request Account Deletion' section to web account page at /app/account

Co-Authored-By: unknown <>
- Add deleteAccount server function using Supabase admin API to delete user
- Update DeleteAccountSection to use actual deletion with confirmation flow
- After deletion, signs out user and redirects to home

Co-Authored-By: unknown <>
- Add service_role_key to SupabaseClient for admin API access
- Add admin_delete_user method using Supabase Admin API (DELETE /auth/v1/admin/users/{id})
- Add DELETE /delete-account route with auth context
- Register route in router and openapi spec

Co-Authored-By: unknown <>
- Add admin_get_stripe_customer_id to look up user's Stripe customer
- Add admin_delete_storage_objects to clean up audio files bucket
- Delete Stripe customer first (cancels subscriptions, webhook syncs stripe schema)
- Delete storage objects (audio-files bucket)
- Delete Supabase auth user last (cascades profiles, nango_connections, transcription_jobs)

Co-Authored-By: unknown <>
Co-Authored-By: unknown <>
Co-Authored-By: unknown <>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Base automatically changed from devin/1771922163-local-first-docs-update to main February 26, 2026 06:38
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