Skip to content

Migrate forgot password page to react component#6792

Merged
thornbill merged 12 commits intojellyfin:masterfrom
MontejoJorge:refactor/forgotPassword-react
May 12, 2025
Merged

Migrate forgot password page to react component#6792
thornbill merged 12 commits intojellyfin:masterfrom
MontejoJorge:refactor/forgotPassword-react

Conversation

@MontejoJorge
Copy link
Copy Markdown
Contributor

@MontejoJorge MontejoJorge commented Apr 21, 2025

Changes

Refactor the forgotPassword page from the legacy controllers to react component.

@MontejoJorge MontejoJorge requested a review from a team as a code owner April 21, 2025 21:38
@jellyfin-bot
Copy link
Copy Markdown
Collaborator

jellyfin-bot commented Apr 21, 2025

Cloudflare Pages deployment

Latest commit 582127a
Status ✅ Deployed!
Preview URL https://f7e9138d.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

Comment thread src/apps/stable/routes/session/forgotPassword/index.tsx Outdated
@MontejoJorge MontejoJorge requested a review from viown April 29, 2025 20:43
Comment thread src/apps/stable/routes/session/forgotPassword/index.tsx Outdated
Comment thread src/strings/es.json Outdated
Comment thread src/apps/stable/routes/session/forgotPassword/index.tsx Outdated
Copy link
Copy Markdown
Member

@thornbill thornbill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I just had a couple minor suggestions. 👍

Comment thread src/apps/stable/routes/session/forgotPassword/index.tsx Outdated
Comment thread src/apps/stable/routes/session/forgotPassword/index.tsx Outdated
Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
@MontejoJorge MontejoJorge requested a review from thornbill May 1, 2025 10:11
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2025

@thornbill thornbill changed the title refactor: forgotPassword page to react component Migrate forgot password page to react component May 12, 2025
@thornbill thornbill added the cleanup Cleanup of legacy code or code smells label May 12, 2025
@thornbill thornbill added this to the v10.11.0 milestone May 12, 2025
@thornbill thornbill merged commit dd38c6b into jellyfin:master May 12, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Cleanup of legacy code or code smells

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants