ProjectOpenSea / opensea-js

TypeScript SDK for the OpenSea marketplace
https://docs.opensea.io/reference
MIT License
2.28k stars 955 forks source link

How well is this SDK maintained? #118

Closed tre-dev closed 1 year ago

tre-dev commented 3 years ago

I'm grateful for the SDK, though I'm having the feeling that it's a bit too unmaintained.

There are several packages (i.e. wyvern-js, wyvern-schemas) that are outdated or bloated due to bad module practice (typescript & webpack as dependencies) https://github.com/ProjectOpenSea/wyvern-schemas/blob/master/package.json#L44

On top of that, I've noticed a few smaller bugs within opensea-js itself (i.e. there are some fields missing when converting from api-json to converted-json)

As a developer, who already integrated OpenSea within my project and is thinking about going even deeper, I need a well-maintained SDK, because when seeing open issues without comments about critical use-cases, it makes me wonder: will it even work when I spend time integrating it?

I hope this makes sense.

0xfoobar commented 3 years ago

The lack of response is an answer in itself

brandsimon commented 3 years ago

Yes, sadly I have the same impression.

ameshkin commented 3 years ago

Yeah having the same issues myself.

omnigolp commented 3 years ago

Since they released a new version, they probably just ignore the issues?

tre-dev commented 3 years ago

There are still quite a lot of issues un-answered. Would be good if they could shift some of their $100M funding into making sure community developers are happy. Starts with transparent communication...

eth0izzle commented 3 years ago

Whilst I agree with you @tre-dev and I'm sure the team are on it but in the meantime you can help the fellow community out by fixing some of the bugs you mentioned and submitting PRs.

tre-dev commented 3 years ago

I would do that if I had the time, but I've got a full schedule and my own projects to work on. :/

Harrison97 commented 3 years ago

Not at all it seems. The readme is completely dead..

tre-dev commented 2 years ago

fyi, I'm moving away from using opensea-js in my frontend code. The size is just too much.

https://bundlephobia.com/package/opensea-js@1.2.1

I wish the OpenSea Team would spend some time & money into it. Maybe they're already working on it, but then the communication isn't really well done.

esteban-OpenSea commented 2 years ago

Hello everyone, we are working diligently to improve the SDK. Regarding the dependencies, I see they are updated. Any other bugs everyone else is encountering?

For future reference, please provide specific details of any bugs/issues you might encounter. We appreciate the general feedback but in order to take actionable steps to fix these, details on the issues would be much appreciated!

esteban-OpenSea commented 2 years ago

Checkin in again, please make sure you are using the most recent version of Node and the SDK.

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

JoshuaSchmidt-OpenSea commented 1 year ago

Version 6 will remove all of the old dependencies. It will only have 3 total dependencies.