-
-
Notifications
You must be signed in to change notification settings - Fork 267
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
Maintainers wanted #114
Comments
I would love to contribute; I'm mainly a TypeScript user and I've used this library quite a bit this past year. I haven't really had experience in creating an API wrapper library, but it's definitely something I'd be interested in (especially since it's Spotify's API)! |
Hi! I would like to help out in any way possible! I ran into this library while working on my capstone project for my bootcamp so I would like to give back and help out in any way I can. My last three projects are with typescript so I’m getting along with it pretty well |
@csantiago132, I've been working on a TypeScript rewrite of this library in my spare time. If you want to help me work on it, I can push my current progress to my personal fork. I think most of the heavy lifting is done, so it's mostly supporting the rest of the endpoints, test coverage, and documentation once finished. |
@adamgrieger yes! Sure I’ll help out in whatever way I can. I’m wrapping up my bootcamp now so I’ll be ready to jump in in a week |
@csantiago132 Sounds good. I just pushed all of my changes to the |
@adamgrieger ok will take a look! ill fork it and start to get to know the codebase |
I added a Discord server link to my fork's README.md where we (and potentially others) can talk about the rewrite. |
@adamgrieger ok, ill update the branch, I went across the code and wow, great job! ill keep digging around and learn the code-base, ill start with seeing the test, i believe thats the easiest way to get to know the codebase |
@adamgrieger @csantiago132 are you guys still actively maintaining? I'd like to try to attack #129 |
I probably won't be maintaining this project anytime in the near future due to work, so it's all yours if you want to take it! |
@lynnpark I'm the only owner so far @adamgrieger, @csantiago132 would you be interested in maintaining this project? |
@JMPerez yes id love to! |
@JMPerez I would love to too, if there is room for one more :) |
Hey, I am wondering what is the state of this case and project. Is it still being maintained? |
If anyone wants to follow my progress, I've been working on an isomorphic wrapper written in TypeScript. Hopefully when it's done, it can serve as an alternative to both |
Just wanted to start a conversation here about a few of these packages mentioned here. I've been hoping to spend some time helping out because I'm using this package in a project. I just want to understand the future of these packages. As far as
It seems like spotify-web-api-node hasn't been touched in a year or two. I know as far as Typescript goes, @adamgrieger has made huge progress on spotify-web-api-ts, which is great because that could be used in place of both
I'd love to hear everyone's thoughts and ideas about all this. |
@BenGu3 Happy to typescript-ify this project, that would help with maintenance. It's painful adding new endpoints due to having to modify many files when one would expect changing the lib + test. If you can explore this approach, I'd be glad to review the changes and make a decision. The overhead for the library output will be minimal. Once the JS is generated it shouldn't be that large compared to the current bundle, and adding a couple of kBs is good trade-off if that makes it easier to keep updated. Regarding @adamgrieger's project, the more the merrier. I can't commit to maintaining more projects at the moment, but I can reach out to the Spotify team to try to list it in their list of wrappers for developers to discover. |
How does this repo relate to https://github.com/thelinmichael/spotify-web-api-node ? When I initially searched for a spotify sdk in nodejs on NPM, that was the version i found. It looks like this repo is what is actually keeping https://www.npmjs.com/package/spotify-web-api-node up to date. But the links to source code on that page link to https://github.com/thelinmichael/spotify-web-api-node. Maybe those links should be updated to point to this project? Will https://github.com/adamgrieger/spotify-web-api-ts be merged into this project in the future? There are several new issues and pull requests being submitted to https://github.com/thelinmichael/spotify-web-api-node but it's not being updated and doesn't get pubished to NPM. IMO it's issues should be closed and/or moved to this project. |
Ah... I think I see what is going on now. Not sure how I missed this before.
Given that my project is in Electron, I'm assuming I want to be using the other project. Though it's confusing to me why https://github.com/thelinmichael/spotify-web-api-node claims:
If it can be used in the browser, and JMPerez is a maintainer of it why does spotify-web-api-js exist? |
We would like to get contributors who would like to maintain and evolve this library
As the main maintainer of this library, I have been busy lately with other projects and I will not able to keep this library up to date for the foreseeable future. I would love to get support from other contributors that have ideas on how to move forward the library so it's still relevant and useful for developers.
If you are interested in maintaining the library, please add a comment to this issue.
The text was updated successfully, but these errors were encountered: