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

At least keep it retro-compatible.... #7

Open
Badisi opened this issue Mar 15, 2021 · 0 comments
Open

At least keep it retro-compatible.... #7

Badisi opened this issue Mar 15, 2021 · 0 comments

Comments

@Badisi
Copy link

Badisi commented Mar 15, 2021

Thanks for the effort and time you've put in this package as it really helped us many times !

But breaking apis in the middle of nowhere without notice is not very friendly and forgetting about retro-compatibility is even worse. We spent hours understanding why everything on our side suddenly brakes to finally find out it was your plugin that was renamed from browsertab to browsertabs. What a pain...

Luckily you kept your git history and what we had to do is simply install the package again from the following commit:
https://github.com/Qbix/cordova-plugin-browsertab.git#92db4d13570706c4e16edb7df313e3cd82e7bd95

But it would be nice to at least add a v0.2.0 tag on that commit and make an announcement of the breaking in the README.

Thanks

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

No branches or pull requests

1 participant