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

Clean-up, Modernize, and Add New "Card Beep" Feature #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shellster
Copy link

@shellster shellster commented Jan 4, 2023

  • Fix WiegandNG code to work with modern NodeMCU chain (ICACHE_RAM_ATTR)
  • Fix Space Information to use String properly (was being interpreted as random bytes in modern build)
  • Fix Code to use latest ArduinoJSON library
  • Fix /api/viewlog to return card data in a useable fashion (Captures aren't just lumped in a weird string number key)
  • Add new /cardbeep feature which uses AJAX to show new captures in near real time with a nice beep (useful to run on phone when doing field captures)
  • Filter Erroneous 2 byte "Card" read that gets detected on boot.
  • Many other minor code cleanups.

@shellster
Copy link
Author

Note: First issue above completely flashing builds of the current repo with a modern Arduino IDE. If you flash them it will softbrick the module.

@shellster
Copy link
Author

If any repo owners respond, I can refactor this into multiple PR's if needed. If I don't hear back in the near future, I'm just going to diverge farther from this branch and maintain my own fork.

@shellster shellster changed the title Initial updates Clean-up, Modernize, and Add New "Card Beep" Feature Jan 6, 2023
@SufficientRole6246
Copy link

SufficientRole6246 commented May 20, 2023

Bit of a noob here. If I want to run your version on my hardware - do I have to load it with the Arduino workflow referenced in the firmware updates section or is there a bin file I can upload via web interface to see/use your version?

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

Successfully merging this pull request may close these issues.

2 participants