-
-
Notifications
You must be signed in to change notification settings - Fork 437
BT Remote: Add Rename Option, simplify Bad KB BLE profile #439
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
Conversation
WillyJL
left a comment
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.
As mentioned in review comment
|
Should I combine these commits into one (or at least remove the separate "revert changes to firmware" commit) and force-push or are "revert" commits okay to be merged into the main repo? |
|
I typically squash merge pull requests, so it doesn't matter anyway :) thanks, will review soon |
Sounds good, thanks! |
Adds an option to rename the advertised Bluetooth device. Closes Next-Flip#410.
Copies some of the firmware code to modify it, rather than directly modifying it in the firmware.
|
also no need to worry about rebasing, it ends up squashed anyway so i merge into pr branches just to test on latest changes and to update the changelog file, and that wont cause issues for squashing. if i have some local changes im yet to push to the pr branch tho, rebasing gets in the way :) |
Sounds good, won't rebase again. I just like seeing a nice tidy graph in VSCode, lol |
|
i think ill edit this a bit more than i wanted to, wanna add a setting file so it remembers, so this profile will need to be moved outside the rename scene and be used from the beginning, similar to badkb. |
Would you rather I do that so you don't have to? I didn't mean to add a ton of work to your plate. |
|
not at all, i want some fun too :D working on it rn |
What's new
Addresses #410.
For the reviewer