search

meshtastic / Meshtastic-Android

Android application for Meshtastic
https://meshtastic.org
GNU General Public License v3.0
748 stars 216 forks source link

fix: Display for self-sent messages #1350

Closed jamesarich closed 1 month ago

jamesarich commented 1 month ago

fixes #1349

This commit updates the getUser function in NodeDB to set the ID of the user when creating a new User object, ensuring that the user ID is always populated.

Additionally, it refactors the Message class to use the proto User object instead of the MeshUser class. This change simplifies the code and improves consistency by utilizing the proto definition for user information.

andrekir commented 1 month ago

Thanks for the effort but the issue here is just a missing setId() in nodeDB.geUser(). If you’re up for it, please update the PR. If not, no worries, I should get to it tomorrow.

jamesarich commented 1 month ago

@andrekir - thank you, i missed that.