feat: add linked accounts page by michaelhthomas · Pull Request #883 · seerr-team/seerr

michaelhthomas · 2024-07-21T18:27:20Z

Description

Adds a "Linked Accounts" page to the user settings, allowing managing of linked media server (Plex, Jellyfin) accounts, including linking an account to your existing local user, or removing a linked account to become a local user. This page doesn't add much new functionality (except the ability to unlink accounts), but is intended to lay the groundwork for a new and improved implementation of #183 which allows the secure linking of OIDC accounts with existing local accounts.

Screenshot (if UI-related)

Plex Linked Accounts Page	Link Account Dropdown

Jellyfin Linked Accounts Page	Link Jellyfin Modal

Emby Linked Accounts Page	Link Emby Modal

To-Dos

Successful build pnpm build
Translation keys pnpm i18n:extract
Database migration (if required)

Issues Fixed or Closed

Fixes #XXXX

michaelhthomas · 2024-07-21T18:30:57Z

Some outstanding questions / TODOs:

Should it be possible to link Plex accounts with a different email address? Right now I'm leaning towards no for simplicity's sake, since otherwise we would have to change how the auth linking works as well.
~~Need to make sure you can't lock yourself out of your account. Particularly, need to make sure that users with a media server account do not unlink it without first setting a password~~
~~Emby? I haven't tested it at all and it currently doesn't have a logo, so will probably just show the jellyfin one~~
Will need help testing this, particularly with Plex since I don't personally use it

gauthier-th

Just a few comments for now.
And you will add Emby once the PR with Jellyfin/Emby as different server types will be merged right?
Otherwise it's great I like it, good job!

michaelhthomas · 2024-07-27T17:59:35Z

And you will add Emby once the PR with Jellyfin/Emby as different server types will be merged right?

Yep, that's the plan currently. Hopefully that PR will make emby a lot easier to integrate.

github-actions · 2024-07-30T16:13:19Z

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

michaelhthomas · 2024-07-30T16:20:51Z

Unified the two dropdown components and addressed a few other issues with the frontend.

michaelhthomas · 2024-08-01T15:48:24Z

Rebased on feat-server-type-setup and added support for emby

github-actions · 2024-08-19T17:25:41Z

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

fallenbagel · 2024-08-20T21:36:23Z

Rebased on feat-server-type-setup and added support for emby

That pr has now been merged

Gauvino · 2024-09-16T20:17:01Z

Why this PR is still in draft any reason for this @michaelhthomas ?

gauthier-th · 2024-09-30T14:34:19Z

Hi @michaelhthomas, could you please rebase this on develop and make it ready for review if everything is ok?

michaelhthomas · 2024-10-18T15:58:26Z

@gauthier-th Rebased on develop. This should be ready for review now, but will probably need some testing. I've personally tested with Jellyfin, Plex, and Emby, but this could definitely use some independent testing.

TheRedCyclops · 2024-11-01T18:18:15Z

Is there a preview tag for testing on docker?

gauthier-th · 2024-11-01T22:02:05Z

Is there a preview tag for testing on docker?

I've just created one: preview-linked-accounts (will be available in ~30min)

gauthier-th

Here are a few more remarks after a detailed review. I still have to test it though.
Feel free to challenge me if you disagree on something.

github-actions · 2025-02-09T23:19:22Z

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

…counts

Adds a shared component for plain dropdown menus, based on the headlessui Menu component. Updates the `ButtonWithDropdown` component to use the same inner components, ensuring that the only difference between the two components is the trigger button, and both use the same components for the actual dropdown menu.

…ounts

Prevents the primary administrator from unlinking their media server account (which would break sync). Additionally, prevents users without a configured local email and password from unlinking their accounts, which would render them unable to log in.

…ecks

fallenbagel

LGTM

fallenbagel · 2025-02-22T16:41:49Z

@all-contributors please add @michaelhthomas for code

allcontributors · 2025-02-22T16:41:58Z

@fallenbagel

I've put up a pull request to add @michaelhthomas! 🎉

* feat(linked-accounts): create page and display linked media server accounts * feat(dropdown): add new shared Dropdown component Adds a shared component for plain dropdown menus, based on the headlessui Menu component. Updates the `ButtonWithDropdown` component to use the same inner components, ensuring that the only difference between the two components is the trigger button, and both use the same components for the actual dropdown menu. * refactor(modal): add support for configuring button props * feat(linked-accounts): add support for linking/unlinking jellyfin accounts * feat(linked-accounts): support linking/unlinking plex accounts * fix(linked-accounts): probibit unlinking accounts in certain cases Prevents the primary administrator from unlinking their media server account (which would break sync). Additionally, prevents users without a configured local email and password from unlinking their accounts, which would render them unable to log in. * feat(linked-accounts): support linking/unlinking emby accounts * style(dropdown): improve style class application * fix(server): improve error handling and API spec * style(usersettings): improve syntax & performance of user password checks * style(linkedaccounts): use applicationName in page description * fix(linkedaccounts): resolve typo * refactor(app): remove RequestError class

fallenbagel · 2025-03-10T18:50:34Z

🎉 This PR is included in version 2.4.0 🎉

The release is available on:

v2.4.0
GitHub release

Your semantic-release bot 📦🚀

* feat(linked-accounts): create page and display linked media server accounts * feat(dropdown): add new shared Dropdown component Adds a shared component for plain dropdown menus, based on the headlessui Menu component. Updates the `ButtonWithDropdown` component to use the same inner components, ensuring that the only difference between the two components is the trigger button, and both use the same components for the actual dropdown menu. * refactor(modal): add support for configuring button props * feat(linked-accounts): add support for linking/unlinking jellyfin accounts * feat(linked-accounts): support linking/unlinking plex accounts * fix(linked-accounts): probibit unlinking accounts in certain cases Prevents the primary administrator from unlinking their media server account (which would break sync). Additionally, prevents users without a configured local email and password from unlinking their accounts, which would render them unable to log in. * feat(linked-accounts): support linking/unlinking emby accounts * style(dropdown): improve style class application * fix(server): improve error handling and API spec * style(usersettings): improve syntax & performance of user password checks * style(linkedaccounts): use applicationName in page description * fix(linkedaccounts): resolve typo * refactor(app): remove RequestError class

michaelhthomas force-pushed the feat/linked-accounts branch from 4f53316 to 7cc7756 Compare July 21, 2024 23:02

gauthier-th reviewed Jul 24, 2024

View reviewed changes

Comment thread server/entity/User.ts

Comment thread src/components/Common/Dropdown/index.tsx

michaelhthomas force-pushed the feat/linked-accounts branch from 7cc7756 to 9da4fbb Compare July 30, 2024 16:13

github-actions Bot added the merge conflict Cannot merge due to merge conflicts label Jul 30, 2024

michaelhthomas force-pushed the feat/linked-accounts branch from 9da4fbb to 4866427 Compare July 30, 2024 16:14

github-actions Bot removed the merge conflict Cannot merge due to merge conflicts label Jul 30, 2024

gauthier-th linked an issue Jul 31, 2024 that may be closed by this pull request

[Feature request] SSO #86

Closed

michaelhthomas force-pushed the feat/linked-accounts branch from cffbbc4 to 72b28f4 Compare August 1, 2024 15:47

michaelhthomas changed the base branch from develop to feat-server-type-setup August 1, 2024 15:47

gauthier-th mentioned this pull request Aug 12, 2024

Roadmap for Seerr #926

Open

github-actions Bot added the merge conflict Cannot merge due to merge conflicts label Aug 19, 2024

lucaam mentioned this pull request Sep 13, 2024

feat: support OIDC authentication #184

Closed

3 tasks

michaelhthomas force-pushed the feat/linked-accounts branch from 72b28f4 to 4a32662 Compare October 18, 2024 15:54

michaelhthomas changed the base branch from feat-server-type-setup to develop October 18, 2024 15:54

michaelhthomas marked this pull request as ready for review October 18, 2024 15:56

michaelhthomas requested a review from fallenbagel as a code owner October 18, 2024 15:56

github-actions Bot removed the merge conflict Cannot merge due to merge conflicts label Oct 19, 2024

gauthier-th reviewed Nov 4, 2024

View reviewed changes

github-actions Bot added the merge conflict Cannot merge due to merge conflicts label Feb 9, 2025

michaelhthomas added 13 commits February 16, 2025 10:25

feat(linked-accounts): create page and display linked media server ac…

29d2ad8

…counts

refactor(modal): add support for configuring button props

e6a22a8

feat(linked-accounts): add support for linking/unlinking jellyfin acc…

5313f05

…ounts

feat(linked-accounts): support linking/unlinking plex accounts

ef9a60c

feat(linked-accounts): support linking/unlinking emby accounts

77f0464

style(dropdown): improve style class application

f0436e5

fix(server): improve error handling and API spec

12554c8

style(usersettings): improve syntax & performance of user password ch…

655efc4

…ecks

style(linkedaccounts): use applicationName in page description

7394e8f

fix(linkedaccounts): resolve typo

6be3343

refactor(app): remove RequestError class

1a88914

michaelhthomas dismissed gauthier-th’s stale review via 1a88914 February 16, 2025 15:31

michaelhthomas force-pushed the feat/linked-accounts branch from 033ccac to 1a88914 Compare February 16, 2025 15:31

github-actions Bot removed the merge conflict Cannot merge due to merge conflicts label Feb 16, 2025

fallenbagel approved these changes Feb 20, 2025

View reviewed changes

gauthier-th approved these changes Feb 20, 2025

View reviewed changes

fallenbagel merged commit 64f05bc into seerr-team:develop Feb 22, 2025

allcontributors Bot mentioned this pull request Feb 22, 2025

docs: add michaelhthomas as a contributor for code #1388

Merged

edbourque0 mentioned this pull request Feb 26, 2025

Add plex home linking #1402

Open

1 task

fallenbagel added the released label Mar 10, 2025

DomiiBunn mentioned this pull request Mar 19, 2025

[Feature request] SSO #86

Closed

Uh oh!

Conversation

michaelhthomas commented Jul 21, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Screenshot (if UI-related)

To-Dos

Issues Fixed or Closed

Uh oh!

michaelhthomas commented Jul 21, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gauthier-th left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

michaelhthomas commented Jul 27, 2024

Uh oh!

github-actions Bot commented Jul 30, 2024

Uh oh!

michaelhthomas commented Jul 30, 2024

Uh oh!

michaelhthomas commented Aug 1, 2024

Uh oh!

github-actions Bot commented Aug 19, 2024

Uh oh!

fallenbagel commented Aug 20, 2024

Uh oh!

Gauvino commented Sep 16, 2024

Uh oh!

gauthier-th commented Sep 30, 2024

Uh oh!

michaelhthomas commented Oct 18, 2024

Uh oh!

TheRedCyclops commented Nov 1, 2024

Uh oh!

gauthier-th commented Nov 1, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gauthier-th left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Feb 9, 2025

Uh oh!

fallenbagel left a comment

Choose a reason for hiding this comment

Uh oh!

fallenbagel commented Feb 22, 2025

Uh oh!

allcontributors Bot commented Feb 22, 2025

Uh oh!

fallenbagel commented Mar 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

michaelhthomas commented Jul 21, 2024 •

edited

Loading

michaelhthomas commented Jul 21, 2024 •

edited

Loading

gauthier-th commented Nov 1, 2024 •

edited

Loading