43081j / id3

A JavaScript ID3 tags parser for Node & browsers.
MIT License
335 stars 63 forks source link

ID3 2.0 Release #27

Closed 43081j closed 5 years ago

43081j commented 9 years ago

I have been gone a long time and have had very little time to maintain this repo (I still do lack time, in fact). However, I am going to give 2.0 a go.

This won't be a release of any new functionality but rather a major cleanup/refactor and improvement of build process, along with any possible bug fixes brought up since I was last around.

TODO

The current branch being used for this is feature/browserify-cleanup, not so appropriately named but it'll do for now.

The browserify output needs testing. I have tried it quickly with an mp3 in my browser and it worked fine.

Please do help if you can, this is not such a simple task and I do not have the capacity to do it alone in an acceptable time frame

J

MaffooBristol commented 9 years ago

Hello! What's the current situation with this release?

43081j commented 9 years ago

We have some issues with the decodeURIComponent calls, I haven't yet had time to go through it.

Also, the fix for #6 seems too much of a workaround to me, rather than a solution. I am not entirely sure why it happens as there seem to be multiple cases. Suppressing it doesn't really tell us why, does it?

vheuken commented 9 years ago

Any progress on this release? Getting #6 fixed is pretty crucial.

r3wt commented 8 years ago

convert readAsBinaryString -> readAsArrayBuffer so we can support IE?

sc1305 commented 8 years ago

Is only the dist/id3.min.js updated with new lib files? Can you please checkin updated id3.js also.