Skip to content

Commit

Permalink
[BUG] App isn't showing message for PDF/file uploads (#2584)
Browse files Browse the repository at this point in the history
* Fixed the issue #2531

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 the condition to check if File Description Exists

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.

* Made the requested Changes

Removed the condition to check for attachment description.
Added the `markdown` inside the touchable and wrapped `attachmentContainer` and the `markdown` inside a `<>` component

* Added file not showing message issue code in this branch

* Fixed the mistake in return

* fix

* Add tests

Co-authored-by: Diego Mello <diegolmello@gmail.com>
  • Loading branch information
saket-m0 and diegolmello authored Nov 4, 2020
1 parent 080b8cc commit 8bc32a3
Show file tree
Hide file tree
Showing 3 changed files with 623 additions and 33 deletions.
Loading

0 comments on commit 8bc32a3

Please sign in to comment.