-
Notifications
You must be signed in to change notification settings - Fork 444
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
Set ScanResponse to off so the advertising will actually advertise... #49
Conversation
Thanks for the PR, before I can accept it, it needs to be tested on Mac OS, Windows, Linux Android and iOS. What operating systems have you tested this with? |
MacOS, iOS and Android. I might be able to help with Windows and Chrome OS (which should cover Linux?) |
Also: irrelevant to the topic, but the latest release zip didn't seem to match the code in the repository. For example the BleKeyboard::onConnected(BLEServer *pServer) was missing. |
I don't know about Chrome OS, but it would be nice to have it tested there as well. I will test it on a standard Linux distro next week and if you or someone else confirms it also works on Windows, I'll accpt the PR. |
Awesome. I will do it on Windows, I can borrow a laptop. |
Normally you would just undo your commit and then make a new commit and force push it. This would automatically modify the PR. E.g. git reset HEAD^
git add .
git commit -m "Set ScanResponse......."
git push --force but if you find that's too complicated, feel free to make another PR. |
Done thanks. I didn't realize it was the last commit in this repository as I had a similar problem in another one. |
I did a quick Linux test and it seems to work. When you've verified it works on Windows, I'll merge. |
All done, looks legit. Thank you, this was smooth. |
Probably yes, but I'm not sure how this has affected iOS/MacOS compatibility. It seems like some of the most recent commits broke compatibility with certain iPhones/iPads and at the same time fix issues for other iPhones/iPads. This kinda makes me wanna drop Apple support completely... |
…the name.
See this line for why.