Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add emoji reactions to message bubbles #1421

Merged
merged 11 commits into from
Dec 3, 2024

Conversation

jamesarich
Copy link
Contributor

@jamesarich jamesarich commented Nov 19, 2024

Adds TapBack functionality to respond to messages via emoji

image

Screen_recording_20241119_174117.webm

closes #1343

Draft - still need to actually collect and send tapbacks.

@caveman99 caveman99 force-pushed the message_emoji_tapbacks branch from 373d1ae to 98989cd Compare November 20, 2024 09:10
Added TapBackEmojiItem composable to display tapback emojis.
Included it in MessageItem composable for incoming messages.
Added a FlowRow to show tapback emojis below the message bubble.
This commit introduces the ability to send and receive emoji tapbacks for messages.

- Adds emoji and replyId fields to DataPacket.
- Adds emoji tapback support to the MeshService
- Modifies UIState to handle emojis in message lists.
Store tapbacks in the database and display them in the message list.
- Add a new table to the database to store tapbacks.
- Add a new DAO method to insert and retrieve tapbacks.
- Update the message list UI to display tapbacks.
@jamesarich jamesarich force-pushed the message_emoji_tapbacks branch from 74b8e36 to 2e82bef Compare November 21, 2024 15:27
@andrekir
Copy link
Member

andrekir commented Dec 1, 2024

@jamesarich are you still working on this? I’d like to wrap it up and get the working parts merged. Let me know if that’s okay with you.

@jamesarich
Copy link
Contributor Author

@jamesarich are you still working on this? I’d like to wrap it up and get the working parts merged. Let me know if that’s okay with you.

@andrekir - I'm on vacation and won't get back to it for a few days yet. Feel free to get it wrapped up and merged in.

@andrekir andrekir changed the title feat: add tapback emojis to message bubbles feat: add emoji reactions to message bubbles Dec 3, 2024
@andrekir andrekir marked this pull request as ready for review December 3, 2024 11:44
@andrekir andrekir merged commit 2234f5a into meshtastic:master Dec 3, 2024
5 checks passed
@jamesarich jamesarich deleted the message_emoji_tapbacks branch December 4, 2024 00:32
@jamesarich
Copy link
Contributor Author

@andrekir thank you!

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.

[Feature Request]: tapbacks/reaction emoji
2 participants