Skip to content

refactor auth to support modal mode - #8142

Open
Prospector wants to merge 4 commits into
mainfrom
prospector/auth-modal
Open

Prospector wants to merge 4 commits into
mainfrom
prospector/auth-modal

Conversation

@Prospector

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

  • Added "Use signed out" to keep your session active but view the site as if you were signed out.

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Frontend previews

Last deployed commit is 6148a68407a668cd1d5218ee08ae072946419e81

Persistent

Environment URL
staging https://pr-8142.frontend-staging.rinth.dev
production https://pr-8142.frontend.rinth.dev
storybook https://pr-8142-storybook.modrinth.workers.dev

Latest commit

Environment URL
staging https://0f1efc23.frontend-staging.rinth.dev
production https://397e7899.frontend.rinth.dev

@clrxbl

clrxbl commented Nov 3, 2026

Copy link
Copy Markdown
Member

@greptileai review

@greptile-apps

greptile-apps Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge.

Summary

Refactors authentication screens so users can sign in, sign up, and reauthenticate in a modal without leaving their current page.

  • Sign-in screens can open over the page someone is viewing.
  • People can sign in or switch accounts without leaving their page.
  • People can create an account in the auth modal.
  • Password reset keeps its page flow in a shared screen.
  • Email verification can run in a shared auth screen.

Reviews (2) · Last reviewed commit: "address rev" · Reviewed by Greptile

@mergify

mergify Bot commented Nov 10, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue). If it does not match the queue conditions yet, it enters the queue once it does.

  • Queue this pull request

@Prospector

Copy link
Copy Markdown
Member Author

@greptileai review

This branch has not been deployed

No deployments
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