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

tx power => txpower (typo) #250

Merged
merged 1 commit into from Mar 2, 2022
Merged

tx power => txpower (typo) #250

merged 1 commit into from Mar 2, 2022

Conversation

aanon4
Copy link
Contributor

@aanon4 aanon4 commented Mar 2, 2022

Typo when setting power using iw command
#249

@ab7pa
Copy link
Contributor

ab7pa commented Mar 2, 2022

@aanon4 Tim, I applied this fix but for some reason the new value does not persist after a reboot. Defaults back to highest Tx Power in the dropdown. Is this a UI issue or something in the backend config save process?

@aanon4
Copy link
Contributor Author

aanon4 commented Mar 2, 2022

My observation with both the old Perl and new Lua was that hitting apply made the value stick until reboot but unless you hit save it would not persist across reboots even with the Perl code. Is this not what you see?

@ab7pa
Copy link
Contributor

ab7pa commented Mar 2, 2022

No, I select a new value from the dropdown list, click APPLY, then click SAVE CHANGES, then click REBOOT. After reboot it has reverted to the old value.

@dman776 dman776 merged commit bdba0c2 into aredn:main Mar 2, 2022
@aanon4
Copy link
Contributor Author

aanon4 commented Mar 3, 2022

Fixed in #252

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants