-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
feat(ui): link processing/requested status badges to service URL #1761
Conversation
4c0c823
to
e925d3b
Compare
68e1f46
to
e015509
Compare
e015509
to
ee50f08
Compare
ee50f08
to
aa6029a
Compare
7bb5ecd
to
fb0a10c
Compare
fb0a10c
to
f27a39a
Compare
I think this should maybe only show for Admins? Just because I give a user permission to manage requests it doesn't mean they necessarily have access to radarr/sonarr. |
Oh hmm, good point! I'll make that change. |
3d03ddf
to
0c6f895
Compare
0c6f895
to
5dbb37e
Compare
e84401d
to
9aba63c
Compare
9aba63c
to
eb86c8c
Compare
🎉 This PR is included in version 1.27.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Description
Similar to how available / partially available status badges link to Plex, make processing / requested status badges clickable links to the *arrs (for users with the
Manage Requests
permission only).Also adds a
url
prop to theBadge
component and refactors theStatusBadge
component to dedupe logic and fix missinginProgress
spinner display for 4K available / partially available media items.Additionally, fixes font weight of media rating values and link hover effects in request cards (both in the Discover page slider & request list). (Originally included in #1832.)
Screenshot (if UI-related)
To-Dos
yarn build
yarn i18n:extract
Issues Fixed or Closed
N/A