Skip to content

Conversation

@matthewCmatt
Copy link
Collaborator

@lukipuki Thanks for working through the bluetooth discovery! This MR is just to keep track of these changes, hopefully we can get things merged up in rust and this MR will need to be updated

@barkincavdaroglu
Copy link
Contributor

@ajmcquilkin looks like systemic change fyi

tauri-plugin-log = { features = ["colored"] , version = "2" }
chrono = { version = "0.4.34", features = ["serde"] }
meshtastic = { git = "https://github.com/meshtastic/rust.git", rev = "6057294aa6cfa979033299f7f02cfe6ceafddeda", features = ["ts-gen", "bluetooth-le"] }
meshtastic = { git = "https://github.com/lukipuki/meshtastic-rust.git", rev = "71978de0121ed87c5953621d7fbe59f25f68fe13", features = ["ts-gen", "bluetooth-le"] }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aspecifically this part @ajmcquilkin

@matthewCmatt
Copy link
Collaborator Author

@barkincavdaroglu Thanks for looking through. for some background, this is a placeholder PR until meshtastic/rust#66 is merged, then this should be switched back to the official rust repo at head.

@lukipuki
Copy link
Contributor

Can you try this new API instead meshtastic/rust#68 ?

I'm also considering to swap out the tuple to a 2-field struct with name: Option<String> and mac_address: BDaddr. It would be great to hear your thoughts on that.

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.

4 participants