Closed SergioGasquez closed 3 months ago
Thanks for the input @Vollbrecht! I think I addressed all the review comments but the UUID one, which can be implemented in a following PR
Thanks for the input @Vollbrecht! I think I addressed all the review comments but the UUID one, which can be implemented in a following PR
Yeah the uuid thing was quite strange ;D can you actually communicate using that hard coded UUID through a public broker with the host-client and the esp? It never worked for me till i changed it.
also one little thing, could you co-author your first commit with juliankrieger julian.krieger@me.com as he did also some of the changes the same way but for 0.48 so we can close https://github.com/esp-rs/std-training/pull/243
can you actually communicate using that hard coded UUID through a public broker with the host-client and the esp?
Didn't tried with a public broker, I did in the past, and it worked, but not recently.
Two more things on the to-do list.
in every .cargo/config.toml -> change idf version to ESP_IDF_VERSION = { value = "tag:v5.2.2" } and remove the panic_imidate_abort for a better .unwrap/() experience.
in every Cargo.toml add a [[bin]] section such that rust-analyzer doesn't topple over building test's like the following.