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

[BREAKING] - Refactor NimBLEHIDDevice #767

Merged
merged 1 commit into from
Dec 8, 2024
Merged

[BREAKING] - Refactor NimBLEHIDDevice #767

merged 1 commit into from
Dec 8, 2024

Conversation

h2zero
Copy link
Owner

@h2zero h2zero commented Nov 29, 2024

  • General code cleanup.
  • All functions that return a characteristic or service have been renamed to have a get prefix to better represent their function.
  • All functions that set a value have been renamed with the prefix set to better represent their functionality.
  • Added an bool notify parameter to setBatteryLevel, which if true, will send a notification of the value to the peer.

@h2zero
Copy link
Owner Author

h2zero commented Nov 29, 2024

@lemmingDev, would you mind testing/reviewing this?

@lemmingDev
Copy link

@sab1e - If you've got the time

* General code cleanup.
* All functions that return a characteristic or service have been renamed to have a `get` prefix to better represent their function.
* All functions that set a value have been renamed with the prefix `set` to better represent their functionality.
* Added an `bool notify` parameter to `setBatteryLevel`, which if `true`, will send a notification of the value to the peer.
@h2zero h2zero merged commit 5ade97a into master Dec 8, 2024
31 checks passed
@h2zero h2zero deleted the refactor-hid branch December 8, 2024 23:40
@lemmingDev
Copy link

@lemmingDev, would you mind testing/reviewing this?

Hi @h2zero

The latest NimBLE-Arduino 2.1.3 is working with ESP32-BLE-Gamepad after a couple of users added some patches to the library

One issue we seem to have now is that on Windows (tested on Windows 10 and 11), it works fine when you pair it the first time, however when you power the board down and it automatically reconnects, gamepad input no longer works.

The main loop seems to be working as any serial output or LEDs modified in the sketch still update.

Android and perhaps Linux seem to work fine still

If you've got the time; any advice on what might need to be changed?

There is a thread with some testing feedback here: lemmingDev/ESP32-BLE-Gamepad#251

Thanks

@h2zero
Copy link
Owner Author

h2zero commented Jan 10, 2025

Thanks @lemmingDev, could you please open an issue about this?

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

Successfully merging this pull request may close these issues.

2 participants