Skip to content

Conversation

@nickvergessen
Copy link
Member

Scenario: Add reaction

  1. Create a conversation
  2. Write a chat message "Message 1"
  3. Add reaction ❤️
  4. Navigate to main page of talk
  5. Assert last message is "You: Message 1" ✅
  6. Open the conversation
  7. Assert last message is "You: Message 1" ❌ see "❤️" instead

Scenario: Remove reaction

  1. Create a conversation
  2. Write a chat message "Message 1"
  3. Add reaction ❤️
  4. Remove reaction ❤️
  5. Navigate to main page of talk
  6. Assert last message is "You: Message 1" ❌ see "You deleted a reaction" instead
  7. Open the conversation
  8. Assert last message is "You: Message 1" ❌ see "You deleted a reaction" instead

Scenario: Remove message

  1. Create a conversation
  2. Write a chat message "Message 1"
  3. Write a chat message "Message 2"
  4. Delete "Message 1"
  5. Navigate to main page of talk
  6. Assert last message is "You: Message 2" ❌ see "You deleted a message" instead
  7. Open the conversation
  8. Assert last message is "You: Message 2" ❌ see "You deleted a message" instead

@nickvergessen nickvergessen added this to the 💛 Next RC (24) milestone Apr 13, 2022
@nickvergessen nickvergessen changed the title Don't update last message of conversation with reaction emoji Don't update last message of conversation with invisible message Apr 13, 2022
@marcoambrosini marcoambrosini merged commit 1361a98 into master Apr 14, 2022
@marcoambrosini marcoambrosini deleted the bugfix/7024/dont-update-last-message branch April 14, 2022 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants