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

[BUG] Fixed the issue #2531 #2541

Closed
wants to merge 2 commits into from
Closed

Conversation

saket-m0
Copy link
Contributor

@saket-m0 saket-m0 commented Oct 9, 2020

Proposed changes

The app now shows the message entered with the pdf or other files on sending.

In app/containers/message/Reply.js added a View Container around the Attachment Touchable and Added a Markdown attribute with msg set to description of attachment to display the message if any.

Issue(s)

Fixing the issue #2531

How to test or reproduce

Screenshots

The text entered while sending the file is visible in the development version now (Refer to the latest attachment sent).

Screenshot_20201010-000013_ DEBUG  RocketChat Experimental

The same text is not visible in the production version (Refer to the latest attachment sent).

Screenshot_20201009-234131_RocketChat

Types of changes

  • [ x ] Bugfix (non-breaking change which fixes an issue)
  • [ x ] Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • [ x ] I have read the CONTRIBUTING doc
  • [ x ] I have signed the CLA
  • [ x ] Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

I am new to Opensource. Please let me know if I have done something wrong so that I can Fix It. Thank You!

In app/containers/message/Reply.js added a View Contaier around the
Attachment Touchable and Added a Markdown attribute with msg set to
description of attachment to display the message if any.
Added an if statement to check if file description exists and if yes
then add a markdown with value msg equal to the description.

Also tested using 'yarn test -u' to add/update the tests.
@saket-m0 saket-m0 closed this Oct 10, 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.

1 participant