-
Notifications
You must be signed in to change notification settings - Fork 574
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
NEW Repo for issues and contributions #509
Comments
@scr2em this plugin creator did fantastic work, and we highly appreciate him , but he is not answering for years and unfortunately it is not maintained anymore. you can install it as an npm package: I dedicate a considerable amount of my free time to developing and maintaining many cordova plugins for the community (See the list with all my maintained plugins). |
@EYALIN Yes, we all highly appreciate him. Did you include how to use it with Capacitor? |
@scr2em I'm not using a capacitor, and not testing it, but some people that used, it said it works. all I can say is that the plunging maintained and tested on Android and IOS in each new big version |
Hi @scr2em, I appreciate your fork and willingness to maintain it! Thank you! |
@dwettstein this plugin creator did fantastic work, and we highly appreciate him , but he is not answering for years and unfortunately it is not maintained anymore. you can install it as an npm package: I dedicate a considerable amount of my free time to developing and maintaining many cordova plugins for the community (See the list with all my maintained plugins). |
Hello @EYALIN, thanks for your response. Unfortunately, it's not possible to use your fork, as you have changed the name and id of the package. This leads to issues when using the Awesome Cordova Plugin wrapper: Additionally your fork is also missing some important fixes, mainly the following PRs:
Thanks for your efforts to keep this project alive! |
@dwettstein the plugin's name does not matter; I'm using Ionic and all works and tested on many devices. if it's not working, please open an issue in my plugin GitHub anyway, @dwettstein I think that giving me a "thumb down" emoji is not respectful of all my efforts for the community (you can see how many plugins I've maintained). |
@EYALIN, sorry that my "thumb down" came off as disrespectful. I gave it because I simply asked user scr2em to merge the PR #460. There was no reason to copy your message about your own fork as an answer to me. I felt like this was kind of advertising your own fork, thus I gave a "thumb down". Of course I appreciate your efforts to the open-source community as well! Thanks. |
@dwettstein Sure, I'll do that tomorrow |
Appreciate everyone's contributions! Just wondering if there is a definitive repo that has all the important changes since this repo stopped being maintained? I understand rebasing and merging all of the different forks/changes is a bit of a nightmare but has anyone done it? |
Hi @jordodev, I ended up having merged/rebased all important fixes (see #509 (comment)) myself in our organisation repo: https://github.com/InfosoftSystems/phonegap-nfc. However, that's not a definitive repo/fork and not published on NPM. Additionally, it may be also updated without announcement, as we use it in our productive app. Feel free to use, fork and contribute. Kind regards. |
@dwettstein Until now I only tested Android. |
Hi,
This repo has not been maintainer for over 4 years, I have contacted the owner to add me as a collaborator or transfer ownership but I didn't get any response.
I have forked it and willing to maintain it and release it to NPM. Probably will be under a different name.
https://github.com/scr2em/phonegap-nfc
If you have any issue please report it here or if you are a contribute any PR is welcome
The text was updated successfully, but these errors were encountered: