Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

Conversation

@udevsharold
Copy link

This is basically a "Copy-Paste" from some of NFCBackground codes for the support of NFCActivator.

@arandomdev
Copy link
Owner

I don't think this fits with the goal of this project, which is to enable native background tag reading. In my opinion, a better course of action would be to update NFCActivator for iOS 13 (or fork it into NFCActivator13 to avoid apt dependencies). The only thing that needs to be done is to find a method in Springboard that receives the NFC event. of course, this should be discussed over there. Furthermore, by updating NFCActivator, we can avoid having 2 event delivery systems.

@arandomdev arandomdev closed this Sep 9, 2020
@udevsharold
Copy link
Author

udevsharold commented Sep 9, 2020

Yeah, that does make sense if you wanted to keep this project as native-background-tag-reading-only.

a better course of action would be to update NFCActivator for iOS 13 (or fork it into NFCActivator13 to avoid apt dependencies)

I did a pull request on NFCActivator, this should probably avoids the dependencies issue: https://github.com/arandomdev/NFCActivator/blob/c1e202d8fe47afdade5be74f1f5753639c4a4474/control#L3
But it's not relevant anymore, since this project won't be implementing the delivery system.

Furthermore, by updating NFCActivator, we can avoid having 2 event delivery systems.

That make sense, if the method does exist. I haven't do much digging on there tbh.

@arandomdev
Copy link
Owner

Thanks for understanding

@arandomdev
Copy link
Owner

Ok did a bit more digging, apparently nfcd only sends out an event when the tag contains a valid URL scheme. So we would have to integrate NFCBackground for NFCActivator to work.

@arandomdev arandomdev reopened this Sep 18, 2020
@arandomdev arandomdev changed the base branch from master to NFCBackground September 20, 2020 21:31
@arandomdev arandomdev changed the base branch from NFCBackground to master September 20, 2020 21:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants