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(new-releases): use hardcoded queryArtistDiscographyAll def #3166

Merged
merged 2 commits into from
Sep 12, 2024

Conversation

ohitstom
Copy link
Contributor

@ohitstom ohitstom commented Sep 12, 2024

Refactor the getArtistEverything function in the new-releases index.js file. Instead of using the Spicetify.GraphQL.Definitions object, the function now directly passes the necessary parameters to the Spicetify.GraphQL.Request function. This change improves code readability and maintainability.

^ first time using the AI commit name thing lol, pretty accurate but we are using the graphql hash because 1.2.46 no longer has this graphql definition

Refactor the getArtistEverything function in the new-releases index.js file. Instead of using the Spicetify.GraphQL.Definitions object, the function now directly passes the necessary parameters to the Spicetify.GraphQL.Request function. This change improves code readability and maintainability.
@ohitstom ohitstom changed the title refactor(new-releases): update getArtistEverything function refactor(new-releases): update getartisteverything function Sep 12, 2024
@ohitstom ohitstom changed the title refactor(new-releases): update getartisteverything function fix(new-releases): update getartisteverything function Sep 12, 2024
@ohitstom ohitstom changed the title fix(new-releases): update getartisteverything function fix(new-releases): use getartisteverything hash Sep 12, 2024
@ohitstom ohitstom changed the title fix(new-releases): use getartisteverything hash fix(new-releases): use hash for artist discography Sep 12, 2024
@rxri
Copy link
Member

rxri commented Sep 12, 2024

run biome

@rxri rxri changed the title fix(new-releases): use hash for artist discography fix(new-releases): use hardcoded queryArtistDiscographyAll def Sep 12, 2024
@ohitstom
Copy link
Contributor Author

ohitstom commented Sep 12, 2024

run biome

i did, i think its failing cause of the title

edit: oh its failing on a file i didnt even change?

@ohitstom ohitstom changed the title fix(new-releases): use hardcoded queryArtistDiscographyAll def fix(new-releases): use hardcoded queryartistdiscographyall def Sep 12, 2024
@ohitstom ohitstom changed the title fix(new-releases): use hardcoded queryartistdiscographyall def fix(new-releases): use hardcoded queryArtistDiscographyAll def Sep 12, 2024
@rxri rxri merged commit 94e5c63 into spicetify:main Sep 12, 2024
6 checks passed
@guiltlab
Copy link

Thank you so much for fixing New Releases, I downloaded the new-releases index.js and put it in my spicetify files, it works again <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants