and refactoring
attribute::update and attribute::report were eating up the error code
returned by set_val and returning ESP_OK. This hide all the errors
reported by set_val.
attribute::update and attribute::report are identical with a simple
delta of whether to call the attribute callback or not. So, refactored
it into as helper function.
attributes
- re-implemented the set_val to set them using the TLV buffer for an
attribute using DataModelProvider::WriteAttribute() API.
- renamed older set_val() to set_val_internal() and made it private.
- changed the set_val's occurances with set_val_internal inside the
component. Since our sdk should not be worrying about getting data
from the internally managed attributes, its safe to use the
set_val_internal().
- updated release notes
Shubham Patil
WanqQixiang
Rohit