Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: cache Jellyfin/Emby avatars from API #1045

Merged
merged 2 commits into from
Oct 24, 2024
Merged

fix: cache Jellyfin/Emby avatars from API #1045

merged 2 commits into from
Oct 24, 2024

Conversation

gauthier-th
Copy link
Collaborator

Description

Previously, avatars were cached using image links from Jellyfin/Emby. Now, avatar images are obtained directly from the API to avoid some configuration bugs.

Screenshot (if UI-related)

To-Dos

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

Issues Fixed or Closed

  • Fixes #XXXX

Previously, avatars were cached using image links from Jellyfin/Emby. Now, avatar images are
obtained directly from the API to avoid some configuration bugs.
@gauthier-th gauthier-th marked this pull request as ready for review October 21, 2024 22:31
@gauthier-th gauthier-th merged commit 0bbcfcb into develop Oct 24, 2024
9 checks passed
@gauthier-th gauthier-th deleted the fix-avatars branch October 24, 2024 16:11
@Panda260
Copy link

Hey since that PR is merged my avatar is no longer availible
image

@Fallenbagel
Copy link
Owner

Hey since that PR is merged my avatar is no longer availible
image

Did you log out and back in? And why are you on develop tag

@Panda260
Copy link

relogin helped
But it would also be good if you didn't have to log in again for the profile picture to be reloaded.

@Fallenbagel
Copy link
Owner

Fallenbagel commented Oct 25, 2024

relogin helped
But it would also be good if you didn't have to log in again for the profile picture to be reloaded.

That's how our system is designed because we don't want to be hitting jellyfin again and again to update avatar (it's not as important as the media) and it only updates when you login.

bonswouar pushed a commit to bonswouar/jellyseerr that referenced this pull request Nov 10, 2024
* fix: cache Jellyfin/Emby avatars from API

Previously, avatars were cached using image links from Jellyfin/Emby. Now, avatar images are
obtained directly from the API to avoid some configuration bugs.

* fix: update avatar on new login
@Fallenbagel
Copy link
Owner

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

thibodelanghe pushed a commit to thibodelanghe/jellyseerr that referenced this pull request Dec 18, 2024
* fix: cache Jellyfin/Emby avatars from API

Previously, avatars were cached using image links from Jellyfin/Emby. Now, avatar images are
obtained directly from the API to avoid some configuration bugs.

* fix: update avatar on new login
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants