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: use copied replied message for showing reply block #663

Conversation

nikkothari22
Copy link
Member

  1. Added field for storing replied message block in a JSON field.
  2. Patch added to migrate all existing messages to new structure
  3. Reply blocks on web and mobile app now use the reply message's JSON instead of fetching the reply message block (perf 🚀). This alone cuts down multiple API calls when chats are loaded.
  4. Reply blocks use the "content" field instead of the HTML field now. (Also perf 🚀). This means that the content would usually fit in a small section (lists would fit in one line for example).
  5. Removed API to get reply message block content.
  6. Removed truncated Tiptap editor on web app since it's not required anymore.

Also closes #608

image

Towards #446 and #472

@nikkothari22 nikkothari22 self-assigned this Feb 9, 2024
@github-actions github-actions bot added enhancement New feature or request Improvement Created by Linear-GitHub Sync linear Medium priority Created by Linear-GitHub Sync 🎨 UI 💻 webapp 🖼️ design 🩻 backend labels Feb 9, 2024
@nikkothari22 nikkothari22 merged commit 20f2cb7 into develop Feb 9, 2024
2 checks passed
@nikkothari22 nikkothari22 deleted the 646-keep-a-copy-of-the-replied-message-instead-of-fetching-and-computing-rendering-of-the-reply-message-block branch February 9, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🩻 backend 🖼️ design enhancement New feature or request Improvement Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync 🎨 UI 💻 webapp
Projects
None yet
1 participant