-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
[FIX] Default Attachments - Show Full Attachment.Text with Markdown #20606
[FIX] Default Attachments - Show Full Attachment.Text with Markdown #20606
Conversation
Unrelated to the PR, you do not need to setup an own bot. RC comes with the |
Thanks for pointing that out :D |
I'll try to test this tomorrow. It's a bit complicated in our 3 instances docker-compose setup, but should work. |
LGTM. |
Seems to fix the issue 👍 |
Why isn't this fix already merged in 3.11.1? |
Emojis and linebreaks are still broken. Yet, Markdown and full size work fine again |
Proposed changes (including videos or screenshots)
Removed truncating of text in
Attachment.Text
.Added
Attachment.Text
to be parsed to markdown by default.Earlier
Now
Issue(s)
Closes #20560
Steps to test or reproduce
#20560 (comment) shows the bug which can be reproduced by using the
cURL
command if you already have a bot with the correct credentials.Otherwise you can use this way to reproduce the message, (I have used a bot for that):
git clone https://github.com/aditya-mitra/rocketchatbot.git && cd rocketchatbot && git checkout reproducing-issue-md && npm install
(this will clone the repo and install dependencies)npm start
.Further comments