-
Notifications
You must be signed in to change notification settings - Fork 51
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
Some remarks #4
Comments
version for node v0.12.x (and v0.11.x) is here: https://github.com/SebastianPozoga/rc522-rfid I rewrite accessor for new v8 engine. But actually it is test version. I will added pull request when I end tests into application. You can test with me ;-) |
Thanks! I'll be using this package on Wednesday and will give your new version a try. |
Hi Sebastian. This are great news. I'll have a look at your version as well and can update mine with your changes. I'll also have a look at my read me file and update it accordingly. |
@rvanderveen in answer to point 3: Node.js application will only work if you run it as root.
PS. @sbrinkmann I created (and tested) a pull request for you ( #5 ). I want add official repository to my service dependencies. It use my fork now. I am hope you approve it soon. Thanks. |
First of all, thank you for writing this package. After tinkering around with it, I got it to work. Some remarks remain - you might put this in the readme so people don't have to search for it as long as I did.
I don't have the knowledge to solve all of the problems, but I'll look into it. Just wanted to report this to help others.
The text was updated successfully, but these errors were encountered: