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

feat(coverAmbience): full rewrite #35

Closed
wants to merge 6 commits into from
Closed

feat(coverAmbience): full rewrite #35

wants to merge 6 commits into from

Conversation

SunsetTechuila
Copy link

@SunsetTechuila SunsetTechuila commented Dec 9, 2023

everything is ready, but the next version of spicetify is required

fixes #18
fixes #31
fixes #33

@SunsetTechuila
Copy link
Author

there are also some type warnings, these will be fixed with the spicetify types update

@SunsetTechuila
Copy link
Author

SunsetTechuila commented Dec 9, 2023

the next version of spicetify is required

rn you can replace the wrapper to test the extension
https://github.com/spicetify/spicetify-cli/blob/master/jsHelper/spicetifyWrapper.js

@SunsetTechuila SunsetTechuila marked this pull request as ready for review December 10, 2023 08:33
@SunsetTechuila
Copy link
Author

@SunsetTechuila SunsetTechuila changed the title refactor(coverAmbience): full rewrite feat(coverAmbience): full rewrite Dec 21, 2023
@SunsetTechuila
Copy link
Author

@Theblockbuster1 hello! you have no plans to merge this, right?

@Theblockbuster1
Copy link
Owner

Hi @SunsetTechuila, I'm sorry about the delays, things got in the way. Thank you for the pull request, even though its closed by now lol. I'm quite unfamiliar with React/TypeScript, so I think that merging it would've been a problem for me to maintain myself anyway. I may get back into things and try implementing your bug fixes into the extension again, I'll see. Thanks again!

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