The notification struct should use Arduino String, not std::string

Smartphone-Companions / ESP32-ANCS-Notifications

Easy-to-use Arduino library for interfacing an ESP 32 with Bluetooth LE ANCS mobile device notifications.

GNU General Public License v3.0

151 stars 24 forks source link

Closed jhud closed 4 years ago

jhud commented 4 years ago

May cause heap issues, but we want to keep this library absolutely simple for beginners.

Internal system should still use std::string.

jhud commented 4 years ago

Fixed in 707c4b4..44742f2 - updated the readme and example, and tested on Hackwatch hardware.