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

NEW Repo for issues and contributions #509

Open
scr2em opened this issue May 16, 2024 · 12 comments
Open

NEW Repo for issues and contributions #509

scr2em opened this issue May 16, 2024 · 12 comments

Comments

@scr2em
Copy link

scr2em commented May 16, 2024

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

@EYALIN
Copy link

EYALIN commented May 16, 2024

@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.
we moved to a maintained plugin (which forked from this one) to here:
https://github.com/EYALIN/community-cordova-plugin-nfc

you can install it as an npm package:
cordova plugin add community-cordova-plugin-nfc --save

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).

@scr2em
Copy link
Author

scr2em commented May 16, 2024

@EYALIN Yes, we all highly appreciate him.

Did you include how to use it with Capacitor?

@EYALIN
Copy link

EYALIN commented May 16, 2024

@scr2em I'm not using a capacitor, and not testing it, but some people that used, it said it works.
but anyway, if not, It will be great if you do PR for it, and I will review and test it.

all I can say is that the plunging maintained and tested on Android and IOS in each new big version

@dwettstein
Copy link

Hi @scr2em, I appreciate your fork and willingness to maintain it!
There is also another fork which has some important fixes: https://github.com/avargaskun/phonegap-nfc/. There is also an open PR in this repo here: #460. Could you please merge this PR as well?

Thank you!

@EYALIN
Copy link

EYALIN commented Jul 19, 2024

@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.
we moved to a maintained plugin (which forked from this one) to here:
https://github.com/EYALIN/community-cordova-plugin-nfc

you can install it as an npm package:
cordova plugin add community-cordova-plugin-nfc --save

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).

@dwettstein
Copy link

dwettstein commented Jul 19, 2024

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:
https://github.com/danielsogl/awesome-cordova-plugins/blob/96678cb14c944c25d92fed990ab41c861362feae/src/%40awesome-cordova-plugins/plugins/nfc/index.ts#L92

Additionally your fork is also missing some important fixes, mainly the following PRs:

Thanks for your efforts to keep this project alive!

@EYALIN
Copy link

EYALIN commented Jul 19, 2024

@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).
even if you have a question or disagree, please respect other people.

@dwettstein
Copy link

@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.

@scr2em
Copy link
Author

scr2em commented Jul 24, 2024

@dwettstein Sure, I'll do that tomorrow

@jordodev
Copy link

jordodev commented Aug 7, 2024

Appreciate everyone's contributions!

Just wondering if there is a definitive repo that has all the important changes since this repo stopped being maintained?
Including Android Background launch bug fix, FLAG_MUTABLE fix, ios fixes, Remove Android Beam etc.

I understand rebasing and merging all of the different forks/changes is a bit of a nightmare but has anyone done it?

@dwettstein
Copy link

dwettstein commented Aug 7, 2024

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.

@brainwind-software
Copy link

@dwettstein
I tried your repo and unfortunately it does not fire and event, when a NDEF tag is read in my ionc & capacitor app.
I switched to https://github.com/ns0m/cordova-plugin-ns0m-nfc for now, which is working for me.

Until now I only tested Android.

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

5 participants