tanishqmanuja / apkmirror-downloader

📥 Download apps from APKMirror with ease.
https://www.npmjs.com/package/apkmirror-downloader
MIT License
35 stars 8 forks source link

Version extraction does not work for weird Twitter version numbering #7

Closed ponces closed 8 months ago

ponces commented 8 months ago

Title is self-explanatory. Twitter version numbering is something like x.x.x-release.x so the last part is taken out during version extraction which makes download to fail. Can this be considered for Twitter? I can push a PR if you prefer but most likely you do this easy-peasy by considering a different regex :P Thanks!

tanishqmanuja commented 8 months ago

Honestly, I would appreciate a PR, my guess is something like '\b\d+(.\d+)+(-\S+)?' should do. Can you test and PR?

tanishqmanuja commented 8 months ago

fixed in commit 8a8be4b57ffee7e45a904a5d922fa525d09d482a

ponces commented 8 months ago

Sorry for not replying but I was trying to fix it on my fork and until now I wasn't able to do so... The same with your last release... image

Did you have success?

tanishqmanuja commented 8 months ago

@ponces, that should be due to latest version being a beta and not having a APK release (it only has a bundle)

ponces commented 8 months ago

You're right! Can't we just ignore alpha and beta tagged releases? I can push an update on that later today if you agree...

tanishqmanuja commented 8 months ago

Nice Idea .. that's exactly what i did for now. But it looks a little bit hacky, I personally use apkmd for downloading a particular version of youtube and youtube-music apks, then generating revanced apps. Thus the hacky solution doesn't bother me enough to implement something better.

But if you implement something better, I will be happy to accept your PR :)

tanishqmanuja commented 8 months ago

ref commit: a4d4826ebd0712db1763d8b67168049357278fd5

ponces commented 8 months ago

Well... My usage is exactly the same but I generate revanced apps for latest version apps ideally :P That's why I need that Twitter latest stable version :) Let me check if your change suits my needs! Thank you very much!!

ponces commented 8 months ago

It's perfectly working! Thanks!