Skip to content

edit and delete actions for thread messages#523

Merged
Francisca105 merged 9 commits intostagingfrom
472-complete-communications
Nov 20, 2025
Merged

edit and delete actions for thread messages#523
Francisca105 merged 9 commits intostagingfrom
472-complete-communications

Conversation

@sancheslfl
Copy link
Copy Markdown
Contributor

Thread messages can now be edited and deleted

Closes #472

@sancheslfl sancheslfl linked an issue Nov 18, 2025 that may be closed by this pull request
@Francisca105 Francisca105 changed the base branch from master to staging November 18, 2025 15:15
Copy link
Copy Markdown
Member

@inesiscosta inesiscosta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At the moment you can only edit/delete outgoing messages. Please replicate this functionality for the other communication types (e.g. outgoing, phone call, meeting, etc.).

Copy link
Copy Markdown
Member

@Francisca105 Francisca105 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Francisca105 Francisca105 dismissed inesiscosta’s stale review November 20, 2025 00:00

Solved in newer commits

@Francisca105 Francisca105 force-pushed the 472-complete-communications branch from 8d470d0 to 1145b54 Compare November 20, 2025 01:33
@Francisca105 Francisca105 force-pushed the 472-complete-communications branch from 1145b54 to de1616f Compare November 20, 2025 01:41
- The icons weren't showing in mobile, now they are always visible (both for pc and mobile);
@Francisca105 Francisca105 merged commit c5b85f2 into staging Nov 20, 2025
1 check passed
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.

Complete communications

3 participants