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

[FIX] When the message is too long declining to send as an attachment does not restore the content into the composer #16332

Merged
merged 13 commits into from
Jun 4, 2020

Conversation

ishriom53tyagi
Copy link
Contributor

[FIX] For bug fixes

Closes #16312 Declining to send a message as an attachment destroys its content

This PR include the new id="textarea" with the help of this we can retrieve the input message.

@ishriom53tyagi ishriom53tyagi changed the title Temp [FIX] Declining to send a message as an attachment destroys its content Jan 26, 2020
@CLAassistant
Copy link

CLAassistant commented Mar 27, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ ggazzo
✅ rodrigok
❌ Danish


Danish seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Kayoku
Copy link

Kayoku commented May 20, 2020

Is it possible to include this in the 3.3 maybe ? I think it's a very important fix.

@rodrigok rodrigok changed the title [FIX] Declining to send a message as an attachment destroys its content [FIX] When the message is too long declining to send as an attachment does not restore the content into the composer Jun 2, 2020
@rodrigok rodrigok added this to the 3.4.0 milestone Jun 2, 2020
@ggazzo ggazzo merged commit 46856e4 into RocketChat:develop Jun 4, 2020
gabriellsh added a commit that referenced this pull request Jun 4, 2020
…rewrite_admin_sidebar

* 'develop' of github.com:RocketChat/Rocket.Chat:
  [FIX] When the message is too long declining to send as an attachment does not restore the content into the composer (#16332)
  [IMPROVE][Federation] Add support for _tcp and protocol DNS entries (#17818)
  Fix the update check not working (#17809)
  Add Apps-Engine to Engine Versions on History (#17810)
  [FIX] Link preview containing HTML encoded chars (#16512)
  [FIX] Email link "go to message" being incorrectly escaped (#17803)
  [FIX] Error when re-installing an App (#17789)
  Update Apps-Engine version (#17804)
  Fix invalid develop payload to release service (#17799)
ggazzo added a commit that referenced this pull request Jun 4, 2020
…into feat/new-threads

* 'feat/new-threads' of github.com:RocketChat/Rocket.Chat:
  Expand/Collapse ThreadView
  [FIX] When the message is too long declining to send as an attachment does not restore the content into the composer (#16332)
@sampaiodiego sampaiodiego mentioned this pull request Jun 30, 2020
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.

[BUG] Declining to send a message as an attachment destroys its content
6 participants