Skip to content

Add support for sending message reactions and expose additional packet metadata#149

Open
skgsergio wants to merge 3 commits into
meshtastic:devfrom
skgsergio:feat/extra-info-and-reactions
Open

Add support for sending message reactions and expose additional packet metadata#149
skgsergio wants to merge 3 commits into
meshtastic:devfrom
skgsergio:feat/extra-info-and-reactions

Conversation

@skgsergio
Copy link
Copy Markdown

Changes:

  • Emoji/Reaction Support: Added is_reaction parameter to send_text, send_direct_message, and broadcast_channel_message services

  • Message Event Metadata:

    • Added via_mqtt, hop_start, and hop_limit properties to Packet class
    • Added hop_start, hop_limit, rx_snr, rx_rssi, and via_mqtt to message_sent events.

Additional info

I noticed when I was going to do the PR with the code already done that my changes cover what #126 proposes.

So, I can drop that part depending on whether we prefer the hop_limit and hop_start or just the number of hops. My two cents: I prefer to have both values and calc the hop numbers if I need it.

@skgsergio
Copy link
Copy Markdown
Author

I also saw this is done in #98 I could close this then 🤷🏻‍♂️

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.

1 participant