Skip to content

[NEW] Send multiple attachments - #2162

Merged
diegolmello merged 76 commits into
developfrom
new.multiple-files
Jun 26, 2020
Merged

diegolmello merged 76 commits into
developfrom
new.multiple-files

Conversation

@djorkaeffalexandre

@djorkaeffalexandre djorkaeffalexandre commented Jun 3, 2020 •

Copy link
Copy Markdown
Contributor

@RocketChat/ReactNative

Closes #1101.

@lgtm-com

lgtm-com Bot commented Jun 3, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 1 when merging a07e6d8 into 594e598 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

# Conflicts:
#	app/containers/MessageBox/index.js
#	app/index.js
#	app/share.js
#	app/views/ShareView/index.js
#	app/views/WithoutServersView.js
#	ios/Podfile.lock
#	ios/Pods/Local Podspecs/RNCAsyncStorage.podspec.json
#	ios/Pods/Local Podspecs/RNImageCropPicker.podspec.json
#	ios/Pods/Manifest.lock
#	ios/Pods/Pods.xcodeproj/project.pbxproj
#	ios/Pods/Target Support Files/Pods-RocketChatRN/Pods-RocketChatRN.debug.xcconfig
#	ios/Pods/Target Support Files/Pods-RocketChatRN/Pods-RocketChatRN.release.xcconfig
#	ios/Pods/Target Support Files/Pods-ShareRocketChatRN/Pods-ShareRocketChatRN.debug.xcconfig
#	ios/Pods/Target Support Files/Pods-ShareRocketChatRN/Pods-ShareRocketChatRN.release.xcconfig
#	ios/Pods/Target Support Files/RNImageCropPicker/ResourceBundle-QBImagePicker-RNImageCropPicker-Info.plist
#	yarn.lock
@lgtm-com

lgtm-com Bot commented Jun 18, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 1 when merging 57ec0ae into a992c51 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

@diegolmello
diegolmello marked this pull request as ready for review June 25, 2020 21:08
@diegolmello diegolmello changed the title [WIP] Send multiple attachments [NEW] Send multiple attachments Jun 25, 2020
@lgtm-com

lgtm-com Bot commented Jun 25, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts and fixes 1 when merging 635e4c4 into a992c51 - view on LGTM.com

new alerts:

  • 1 for Potentially inconsistent state update
  • 1 for Unsupported state update in lifecycle method

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

@lgtm-com

lgtm-com Bot commented Jun 26, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts and fixes 1 when merging bad484c into a992c51 - view on LGTM.com

new alerts:

  • 1 for Potentially inconsistent state update
  • 1 for Unsupported state update in lifecycle method

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

Comment thread app/views/ShareListView/index.js Outdated
Comment thread app/views/ShareView/Thumbs.js Outdated
Comment thread app/selectors/login.js Outdated
Comment thread app/views/ShareView/utils.js Outdated
Comment thread app/views/AttachmentView.js Outdated
Comment thread app/views/AttachmentView.js
Comment thread app/containers/MessageBox/RightButtons.android.js
Comment thread app/views/ShareView/index.js
Comment thread app/views/ShareView/index.js
Comment thread app/views/ShareView/index.js Outdated
@lgtm-com

lgtm-com Bot commented Jun 26, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts and fixes 1 when merging 2b7163c into a992c51 - view on LGTM.com

new alerts:

  • 1 for Potentially inconsistent state update
  • 1 for Unsupported state update in lifecycle method

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

@lgtm-com

lgtm-com Bot commented Jun 26, 2020

Copy link
Copy Markdown

This pull request introduces 4 alerts and fixes 1 when merging e2739d2 into a992c51 - view on LGTM.com

new alerts:

  • 2 for Useless conditional
  • 1 for Potentially inconsistent state update
  • 1 for Unsupported state update in lifecycle method

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

@lgtm-com

lgtm-com Bot commented Jun 26, 2020

Copy link
Copy Markdown

This pull request introduces 2 alerts and fixes 1 when merging 7693629 into a992c51 - view on LGTM.com

new alerts:

  • 1 for Potentially inconsistent state update
  • 1 for Unsupported state update in lifecycle method

fixed alerts:

  • 1 for Unsupported state update in lifecycle method

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement] Multiple Uploads

2 participants