Skip to content

Rename Sign-in URL to Initiate login URI in README - #122

Merged
joyashin merged 2 commits into
mainfrom
devin/1789408020-initiate-login-uri-docs
Sep 14, 2026
Merged

joyashin merged 2 commits into
mainfrom
devin/1789408020-initiate-login-uri-docs

Conversation

@joyashin

Copy link
Copy Markdown
Contributor

Summary

Updates the README to match the WorkOS dashboard, which labels this setting Initiate login URI (the initiate_login_uri field) and exposes it on each application's Redirects tab.

  • Step "3. Create Sign-in URL" → "3. Create Initiate Login URI" (anchor #3-create-initiate-login-uri), plus prose and the dashboard configuration checklist
  • Dashboard instructions now point to the application's Redirects tab

Docs-only change. Related: workos/authkit-nextjs#483.

Link to Devin session: https://app.devin.ai/sessions/7e8691c5bdb94045802f2a2ac7df233b
Open in Devin Desktop: https://app.devin.ai/desktop/session/7e8691c5bdb94045802f2a2ac7df233b?variant=devin
Requested by: @joyashin

Match the WorkOS dashboard, which labels this setting "Initiate login
URI" under an application's Redirects tab.
@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. Add '(aside)' to your comment to have me ignore it.
  • 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, CI, and merge conflict monitoring

Original prompt from julie

SYSTEM:
<latest_message>
Julie Shin (U088X9Q0RL7) [ts=1789407581.624509]: @Devin Can you fix this?
</latest_message>

=== BEGIN THREAD HISTORY (in #team-authkit) ===
Julie Shin (U088X9Q0RL7) [ts=1783704009.704169]: Hey team :wave_:

Naming inconsistencies across redirects has come up time and time again, so I think it would be great to settle on some simplified naming. Please give it a look through and let me know if there are any objections to the following naming proposal...

OAuth allowlists (validated, multi-value, one default) keep the spec term "URIs"; single-link settings use "URL":

• Redirect URIs (Unchanged)
• Sign-out redirects &gt; Sign-out URIs
• Sign-in endpoint &gt; Sign-in URL
• App homepage &gt; App homepage URL
• Sign-up URL (Unchanged)
• User invitation URL (Unchanged)
• Password reset URL (Unchanged)

Kesin Ryan Dehejia (U0A1ZEBT457) [ts=1783705021.853499]: I like all of these changes

Cameron Matheson (U04RVSX5FCK) [ts=1784046590.003779]: we've talked about this before but i hate the "sign-in endpoint/url" terminology (should be "initiate login"). here is an example of a customer who was confused by our weirdness:

(we generally like "sign in" over "login", but this is a standard idea that we shouldn't mess with)

<https://work-os.slack.com/archives/C0BHSSNAMR6/p1784029240707889|https://work-os.slack.com/archives/C0BHSSNAMR6/p1784029240707889>

[Slack unfurl — this is an automatic link preview, not a user message]
Quote of conversation (https://work-os.slack.com/archives/C0BHSSNAMR6/p1784029240707889):
> From Gavin Fogarty
> Hi team 👋 hoping you can point me at one dashboard setting I can't find.

> Setup: We're on AuthKit in our Staging environment ("SocialTalent's Project", application "Cara 2.0 Beta", client ID client_01KXDRR8QCMW64VE7V5ZT2CQ7S). We use a self-hosted sign-in page (built with authkit-nextjs) at &lt;https://carabeta.app/sign-in&gt; — not the WorkOS-hosted one.

> The problem: Sign-ins that start from our own ... (3724 chars truncated...)

@joyashin
joyashin marked this pull request as ready for review September 14, 2026 17:49
@greptile-apps

greptile-apps Bot commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The documentation-only change appears safe to merge.

Summary

  • Renames the setup heading and related references.
  • Updates the troubleshooting anchor and dashboard checklist.
  • Clarifies navigation through Applications to the Redirects tab.

Reviews (2) · Last reviewed commit: "docs: Clarify dashboard path for Initiat..."

@joyashin
joyashin merged commit 56adeb6 into main Sep 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants