-
Notifications
You must be signed in to change notification settings - Fork 50
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
Extension stuck loading for songs without lyrics #966
Comments
I have this same issue (MacOS Sonoma 14.1) as well. This occurs on songs without lyrics, but also songs without synced lyrics |
I think it's happening with non-synced lyrics for me too actually. |
It must've happened due to the server returning a 500 Errror status Code. Let me explain. |
+1 same issue for me, multiple computers and networks |
I mean you could edit the actual built file for beautiful lyrics and do all that, but If you are a dev, you could just make your own beautiful lyrics. (im in the middle of doing that) |
Could you make an option to turn off the background colors in the Now
Playing section?
…On Tue, Nov 5, 2024 at 1:39 PM SpikeNew7774 ***@***.***> wrote:
I mean you could edit the actual built file for beautiful lyrics and do
all that, but If you are a dev, you could just make your own beautiful
lyrics. (im in the middle of doing that)
—
Reply to this email directly, view it on GitHub
<#966 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A5MWCYFOYWTSKA5PTCTENU3Z7ENGFAVCNFSM6AAAAABRDMSXRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGAYTAOBXGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
@zero-mindset Do you mean that in the Beautiful-Lyrics extension or in the one that I'm making (I'm thinking of publishing it later) |
Either or. I just figured you’d take what was existing and make tweaks to
that. To me, that’s the only flaw with the current BL design
…On Tue, Nov 5, 2024 at 2:19 PM SpikeNew7774 ***@***.***> wrote:
@zero-mindset <https://github.com/zero-mindset> Do you mean that in the
Beautiful-Lyrics extension or in the one that I'm making (I'm thinking of
publishing it later)
—
Reply to this email directly, view it on GitHub
<#966 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A5MWCYGJHMI6A2A65AR4GL3Z7ER4VAVCNFSM6AAAAABRDMSXRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGA3TQOJVHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I could have a look into that, but if you would want it, it would need to be externally installed and not through the marketplace. I'm probably going to edit the actual output file and not the source code in this github repo -> (PS: Its tooooooooooo complicated for me). |
Anyone find a fix for this? |
Kind of. Its a problem on the server, which could be fixed by adding a check if the response status isnt 200 to just hide the lyrics in the actual client code. Also I found out that if the lyrics are static on apple music, it will return a 500 error with the content about a 429 error. Response Body: "Stage: AM_S, Error: 429 | Too Many Requests" |
I've got same issue for few days for almost all lyrics tho( |
That means their API must be failing. I'm currently making my own Apple Musix => JSON API that's compatible with the beautiful-lyrics format (I'm using it in my own Lyrics extension) |
How did you install Beautiful Lyrics?
Marketplace
What is the Bug/Issue?
The extension loads infinitely in both the sidebar and fullscreen mode when encountering a song without lyrics.
What Country are you from?
Canada
What Operating System are you using?
Windows
Spotify/Spicetify Information
Spotify Version: 1.2.49.439.gfae492c7
Spotify Runtime: cef_129.0.11+g57354b8+chromium-129.0.6668.90
Spicetify Version: 2.38.4
Spicetify Theme: marketplace
Spicetify Extensions: [beautiful-lyrics.js, hidePodcasts.js, goToSong.js, fullAlbumDate.js, playNextWrapper.js, imageOpener.js, CoverAmbience.js, oneko.js, copytoclipboard.js, made-for-you-shortcut.js, playbarClock.js, showQueueDuration.js, spotispy.js, shuffle+.js, AnonymizedRadios.js, cat-jam.js, beautiful-lyrics.mjs]
Spicetify Custom Apps: [marketplace]
Spotify HTML/CSS and Console Screenshots
Spotify.zip
Information Checklist
The text was updated successfully, but these errors were encountered: