Skip to content

Commit

Permalink
fix(wrapper): modify the keywords to find playlistMenuChunk (#2762)
Browse files Browse the repository at this point in the history
  • Loading branch information
rxri authored Jan 11, 2024
1 parent 80fdd4a commit 9a6ecbe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion biome.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"$schema": "https://biomejs.dev/schemas/1.5.0/schema.json",
"$schema": "https://biomejs.dev/schemas/1.5.1/schema.json",
"organizeImports": {
"enabled": true
},
Expand Down
2 changes: 1 addition & 1 deletion jsHelper/spicetifyWrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,7 @@ window.Spicetify = {
});

const playlistMenuChunk = Object.entries(require.m).find(
([, value]) => value.toString().includes('value:"playlist"') && value.toString().includes("contextmenu")
([, value]) => value.toString().includes('value:"playlist"') && value.toString().includes("canView") && value.toString().includes("permissions")
);
if (playlistMenuChunk)
Spicetify.ReactComponent.PlaylistMenu = Object.values(require(playlistMenuChunk[0])).find(m => typeof m === "function" || typeof m === "object");
Expand Down

5 comments on commit 9a6ecbe

@rxri
Copy link
Member Author

@rxri rxri commented on 9a6ecbe Jan 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sitiom do you perhaps know why winget release action fails? First time seeing this error

@sitiom
Copy link
Contributor

@sitiom sitiom commented on 9a6ecbe Jan 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's because https://github.com/spicetify/winget-pkgs is outdated by a week. Try syncing it manually and rerun the workflow again.

@rxri
Copy link
Member Author

@rxri rxri commented on 9a6ecbe Jan 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh yea, that was it haha. i thought it will not make a problem since last time i think this fork was even more outdated than it was now

@sitiom
Copy link
Contributor

@sitiom sitiom commented on 9a6ecbe Jan 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spicetify/winget-pkgs@93f07cf Merge commit? Should have just used the sync button to fast-forward it

@rxri
Copy link
Member Author

@rxri rxri commented on 9a6ecbe Jan 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah probably. I told him to merge the pull request. I kinda forgot that this button exists, oopsie

Please sign in to comment.