Skip to content

Restore sign-up setting descriptions#1469

Merged
N2D4 merged 3 commits into
devfrom
devin/1779411003-restore-signup-hints
May 22, 2026
Merged

Restore sign-up setting descriptions#1469
N2D4 merged 3 commits into
devfrom
devin/1779411003-restore-signup-hints

Conversation

@N2D4
Copy link
Copy Markdown
Contributor

@N2D4 N2D4 commented May 22, 2026

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 Methods sign-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

  • Redesigned OAuth account merge strategy labels with clearer, user-friendly terminology including "Link accounts", "Create new account", and "Block sign-up" options
  • Enhanced sign-up configuration options with additional guidance text explaining email verification requirements and account creation behavior
  • Renamed multi-provider mode dropdown to "Same-email social login policy" with updated help text for improved clarity

Review Change Stack

Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a7d53ffd-bb7e-4b03-85e6-d6133ddb823e

📥 Commits

Reviewing files that changed from the base of the PR and between d12968e and 8cf54a4.

📒 Files selected for processing (1)
  • apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/auth-methods/page-client.tsx

📝 Walkthrough

Walkthrough

Updated 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.

Changes

OAuth and Sign-up Configuration UI Text Updates

Layer / File(s) Summary
OAuth merge strategy display labels
apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/auth-methods/page-client.tsx
MERGE_STRATEGY_SHORT entries updated to new user-facing text: link_method → "Link accounts", allow_duplicates → "Create new account", raise_error → "Block sign-up".
Sign-up section hints and dropdown label
apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/auth-methods/page-client.tsx
"Allow new user sign-ups" toggle gained explanatory hint; "Require email verification" hint updated to clarify restriction until primary email verification; dropdown label/help text changed to "Same-email social login policy" with new explanatory copy.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🐰 I hopped through labels, light and quick,

I nudged the text, a gentle lick,
"Link accounts" and "Block sign-up" gleam,
Hints that guide like carrot beam,
Dashboard now reads smooth and slick.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch devin/1779411003-restore-signup-hints

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.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
stack-auth-hosted-components Ready Ready Preview, Comment May 22, 2026 1:07am
stack-auth-mcp Ready Ready Preview, Comment May 22, 2026 1:07am
stack-auth-skills Ready Ready Preview, Comment May 22, 2026 1:07am
stack-backend Ready Ready Preview, Comment May 22, 2026 1:07am
stack-dashboard Ready Ready Preview, Comment May 22, 2026 1:07am
stack-demo Ready Ready Preview, Comment May 22, 2026 1:07am
stack-docs Ready Ready Preview, Comment May 22, 2026 1:07am
stack-preview-backend Ready Ready Preview, Comment May 22, 2026 1:07am
stack-preview-dashboard Ready Ready Preview, Comment May 22, 2026 1:07am

devin-ai-integration Bot and others added 2 commits May 22, 2026 00:57
…y' with updated options and description

Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>
@N2D4 N2D4 marked this pull request as ready for review May 22, 2026 00:58
Copilot AI review requested due to automatic review settings May 22, 2026 00:58
@N2D4 N2D4 merged commit 9fc7332 into dev May 22, 2026
21 of 37 checks passed
@N2D4 N2D4 deleted the devin/1779411003-restore-signup-hints branch May 22, 2026 00:58
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 22, 2026

Greptile Summary

This 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.

  • Adds hint props to the "Allow new user sign-ups" and "Require email verification" MethodToggleRow components, matching the existing pattern used by other toggles in the file.
  • Renames the merge-strategy dropdown short labels ("Link providers" → "Link accounts", "Allow duplicates" → "Create new account", "Block multiple" → "Block sign-up") and replaces the section title "Multi-provider sign-up mode" with "Same-email social login policy", adding an inline description sentence.

Confidence Score: 5/5

Safe 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

Filename Overview
apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/auth-methods/page-client.tsx Restores hint descriptions for sign-up toggles, renames merge-strategy short labels, and renames the multi-provider setting label with an inline description.

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"]
Loading

Reviews (1): Last reviewed commit: "fix: update same-email social login poli..." | Re-trigger Greptile

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