Skip to content

refactor auth to support modal mode - #8142

Merged
Prospector merged 4 commits into
mainfrom
prospector/auth-modal
Nov 10, 2026
Merged

Prospector merged 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

This pull request is merged.

@Prospector

Copy link
Copy Markdown
Member Author

@greptileai review

@Prospector
Prospector merged commit 0aee4fb into main Nov 10, 2026
16 of 17 checks passed
@Prospector
Prospector deleted the prospector/auth-modal branch November 10, 2026 20:42
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