Restore sign-up setting descriptions#1469
Conversation
Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated UI copy in the auth methods dashboard: OAuth merge strategy labels now show action-oriented text; sign-up toggles received explanatory hint text; the social-login dropdown label/help text was changed to "Same-email social login policy" with new explanatory copy. ChangesOAuth and Sign-up Configuration UI Text Updates
🎯 1 (Trivial) | ⏱️ ~3 minutes
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…y' with updated options and description Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
Greptile SummaryThis PR restores descriptive hint text for the three sign-up settings that were lost during the auth-app redesign, and lightly improves the copy for the OAuth account-merge strategy selector.
Confidence Score: 5/5Safe to merge — changes are purely UI copy updates with no logic modifications. All changes are text additions and label renames: hint strings wired into an existing, already-used prop on MethodToggleRow, a section title rename, and updated short labels for the merge-strategy dropdown. No logic, state, or data-flow is altered. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Sign-up Settings Card] --> B[Allow new user sign-ups toggle]
A --> C[Require email verification toggle]
A --> D[Same-email social login policy dropdown]
B -->|hint added| B1["Existing users can still sign in when sign-up is disabled. You can always create new accounts manually via the dashboard."]
C -->|hint added| C1["Users must verify their primary email before they can use your application. Unverified users will be restricted."]
D --> E{Merge Strategy}
E -->|link_method| E1["Link accounts"]
E -->|allow_duplicates| E2["Create new account"]
E -->|raise_error| E3["Block sign-up"]
D -->|description added| D1["Determines what happens when a user uses a new social login provider with an email that's already connected to an account"]
Reviews (1): Last reviewed commit: "fix: update same-email social login poli..." | Re-trigger Greptile |
Re-adds the hint descriptions for the three sign-up settings that were removed during the auth app redesign, minus the SDK version requirement info.
Link to Devin session: https://app.devin.ai/sessions/a7b91e26dfa843aabc4c0068312609af
Requested by: @N2D4
Note
Low Risk
Low risk UI text-only changes to the dashboard auth settings; no auth/config logic is modified.
Overview
Updates the dashboard
Auth Methodssign-up settings UI to restore descriptive hints for allow sign-ups and require email verification.Renames the multi-provider sign-up mode control to Same-email social login policy, adds explanatory helper text, and revises the merge-strategy option labels (e.g., “Link accounts”, “Create new account”, “Block sign-up”) to better reflect behavior.
Reviewed by Cursor Bugbot for commit 8cf54a4. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit
Improvements