-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
SubGhz some improvements #465
Conversation
return true; | ||
}, | ||
true); | ||
subghz_transmitter->callback( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can add a check here so that other buttons don't send the original signal. This is useful if there are no custom buttons in the protocol.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, all seems to be working fine, I made couple changes to fix issues I found
What's new
SubGhz app: removed duplicate code in
transmitter.c
SubGhz app: fix "add manual" Alutech AT 4N
SubGhzRemote app: drop include from SubGhz app
SubGhz: Improved code readability and some "if" optimization in custom button and keeloq
Fix strange bug with formatting in VS CODE in
keeloq.c
andstar_line.c
Verification
Checklist (For Reviewer)