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

Integration WAX630 #21

Closed
RickDG2 opened this issue Feb 8, 2024 · 2 comments
Closed

Integration WAX630 #21

RickDG2 opened this issue Feb 8, 2024 · 2 comments

Comments

@RickDG2
Copy link
Contributor

RickDG2 commented Feb 8, 2024

Support for wlan2 for WAX630.

These minor changes are suggested in order to support wlan2 on the WAX630 (the lines to change should be pretty obvious, near each occurrence of wlan1).

sensor.py:

for lan in ["wlan0", "wlan1", "wlan2"]:

for lan in ["lan", "wlan0", "wlan1", "wlan2"]:

client_wax.py:

for lan in ["lan", "wlan0", "wlan1", "wlan2"]:

const.py:

           },
           "wlan2": {
               "traffic": "",
               "channelUtil": "",

Those seem to be enough to properly support my WAX630.

@RickDG2
Copy link
Contributor Author

RickDG2 commented Feb 8, 2024

Made it even easier. Learned all I needed to know to use command line git to clone, commit, push and pull request the eight lines of changes. Only took me an hour to learn. I was an old Subversion user for decades.

The Pull Request is pending review, etc.

Yay, me, first open source contribution.

I'll close this request and let the pull request work its way through.

@RickDG2 RickDG2 closed this as completed Feb 8, 2024
@RickDG2
Copy link
Contributor Author

RickDG2 commented Feb 8, 2024

See Pull Request #22.

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

1 participant