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

Follow-up(Edit message) #11451

Merged
merged 4 commits into from
Jan 31, 2024
Merged

Follow-up(Edit message) #11451

merged 4 commits into from
Jan 31, 2024

Conversation

DorraJaouad
Copy link
Contributor

@DorraJaouad DorraJaouad commented Jan 25, 2024

☑️ Resolves

Remark
Mention hint when editing ( NcNoteCard ) will be shown if the message has @ initially (before editing it). Otherwise, we will need to compute diff every time the text is edited (not worth it and can be large for long messages)

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
image image

Newly added
image

🚧 Tasks

  • Visual review
  • Code review

🏁 Checklist

@DorraJaouad DorraJaouad added enhancement feature: chat 💬 Chat and system messages labels Jan 25, 2024
@DorraJaouad DorraJaouad added this to the 💞 Next Major (29) milestone Jan 25, 2024
@DorraJaouad DorraJaouad self-assigned this Jan 25, 2024
@DorraJaouad DorraJaouad force-pushed the feat/11206/edit-message-mentions branch from 3315110 to de32f0b Compare January 26, 2024 20:55
@DorraJaouad DorraJaouad marked this pull request as ready for review January 26, 2024 20:57
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

Nice in general, some technical nitpicks:

Signed-off-by: DorraJaouad <dorra.jaoued7@gmail.com>
Signed-off-by: DorraJaouad <dorra.jaoued7@gmail.com>
@DorraJaouad DorraJaouad force-pushed the feat/11206/edit-message-mentions branch from de32f0b to c2a087b Compare January 30, 2024 10:27
src/utils/textParse.js Outdated Show resolved Hide resolved
@DorraJaouad DorraJaouad force-pushed the feat/11206/edit-message-mentions branch 5 times, most recently from 21b37c3 to f6e7f1d Compare January 31, 2024 16:33
Signed-off-by: DorraJaouad <dorra.jaoued7@gmail.com>
Signed-off-by: DorraJaouad <dorra.jaoued7@gmail.com>
@DorraJaouad DorraJaouad force-pushed the feat/11206/edit-message-mentions branch from f6e7f1d to c0d7bad Compare January 31, 2024 16:41
@DorraJaouad DorraJaouad merged commit 2bf22e1 into main Jan 31, 2024
36 checks passed
@DorraJaouad DorraJaouad deleted the feat/11206/edit-message-mentions branch January 31, 2024 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: chat 💬 Chat and system messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Follow-up(Edit message)
3 participants