Skip to content

Feature: Player Interactions#52

Open
saeidalz13 wants to merge 6 commits into
mainfrom
msg_sending_featuer
Open

Feature: Player Interactions#52
saeidalz13 wants to merge 6 commits into
mainfrom
msg_sending_featuer

Conversation

@saeidalz13
Copy link
Copy Markdown
Owner

@saeidalz13 saeidalz13 commented Jul 8, 2024

Overview

This PR introduces the feature of player interaction which takes care of the communication (sending messages or emojis) between players.

Technical Features

  1. Added CodePlayerInteraction for player interaction signal.
  2. Added PlayerInteraction struct as the payload for messaging.
  3. Wrote unit tests for this feature.

@saeidalz13 saeidalz13 requested a review from mori-ahk July 8, 2024 20:01
@saeidalz13 saeidalz13 self-assigned this Jul 8, 2024
@saeidalz13 saeidalz13 force-pushed the msg_sending_featuer branch from 21e4594 to 759db5d Compare July 16, 2024 20:24
@saeidalz13
Copy link
Copy Markdown
Owner Author

Wooops!! I rebased this with restructure branch and now it has a lot of commits

@mori-ahk mori-ahk force-pushed the msg_sending_featuer branch from 03debbc to afdb9df Compare July 18, 2024 23:48
@saeidalz13
Copy link
Copy Markdown
Owner Author

Should I merge to staging?

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.

2 participants