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

Fixes for Arcadyan Router plus minor additions #13

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

Conversation

zoopra9457
Copy link

@zoopra9457 zoopra9457 commented Apr 19, 2022

Three Changes:

  • I found that the program incorrectly listed the LTE band for the 5G band.
  • Added a Dockerfile if someone wants to generate a docker image with the App. I run this in a Synology NAS at my lake house so I can monitor remotely.

BEFORE ACCEPTING -- Third Change:
Since I am using this to monitor my Arcadyan router remotely, I don't have access to the T-Mobile App, so I wanted a way to monitor the firmware version, so I added that to the display. The issue is I don't have access to a Nokia to correctly identify the Firmware version. This will not work correctly with the Nokia. So if someone can give me the code to get the Firmware of the Nokia version, I'll incorporate it. I think that would be important before accepting this PR

Screen Shot 2022-04-18 at 7 01 41 PM

Also, I can change this to merge into a different branch in the origin repo if someone can create it for me.

I'm actually debating about create a Prometheus Exporter of this data. I could add that to this program or just create my own. Let me know your thoughts.

@highvolt-dev
Copy link
Owner

Sorry to keep you waiting on this PR, this is on my daily list to loop back to and I have not had time, but I do want to incorporate your work!

@zoopra9457
Copy link
Author

No worries, @highvolt-dev! I've got it working in my environment and that's good enough for me :)

I did start work on a prometheus exporter. Haven't published it but if you have interest in helping with the Nokia implementation, I'll get it out there.

@tarunVreddy
Copy link

Please merge this in @highvolt-dev. Critical fix for showing proper band on 5G for Arcadyan router

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.

3 participants