Skip to content

[NEW] Send multiple attachments#2162

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

[NEW] Send multiple attachments#2162
diegolmello merged 76 commits intodevelopfrom
new.multiple-files

Conversation

@djorkaeffalexandre
Copy link
Copy Markdown
Contributor

@djorkaeffalexandre djorkaeffalexandre commented Jun 3, 2020 •
edited by diegolmello
Loading

@RocketChat/ReactNative

Closes #1101.

@djorkaeffalexandre
[CHORE] Update react-native-image-crop-picker
8810543
@djorkaeffalexandre
[CHORE] Update rn-extensions-share (2.4.0-rc.0)
10e549e
@djorkaeffalexandre
[IMPROVEMENT] withoutServersView -> showErrorAlert
0144565
@djorkaeffalexandre
[TEMP] Cherry pick first item
093efff
@djorkaeffalexandre
[CHORE] Update rn-extensions-share (2.4.0-rc.1)
0820d0c
@djorkaeffalexandre
[IMPROVEMENT] Remove some fetch-blob uses
3f33779
@djorkaeffalexandre
[WIP] Initial send multiple files logic
2748591
@djorkaeffalexandre
[WIP] Made images switchable
264aef4
@djorkaeffalexandre
[WIP] Some share extension improvements
073832f
@djorkaeffalexandre
[WIP] Add error styles
41ef5a9
@djorkaeffalexandre
[WIP] Sending multiple files
5bbb106
@djorkaeffalexandre
[WIP] Add video thumbnails
a960723
@djorkaeffalexandre
[WIP] Show video component
6a733d6
@djorkaeffalexandre
[WIP] Share text
31c09a0
@djorkaeffalexandre
[WIP] Send text message (Share Extension)
2af59b8
@djorkaeffalexandre
[WIP] Show list just when have more than one attachment
fa14705
@djorkaeffalexandre
[WIP] Refactor code
88a91f7
@djorkaeffalexandre
[WIP] Add some comments
c276153
@djorkaeffalexandre
[WIP] Fix thumbs height
fb9feb5
@djorkaeffalexandre
[WIP] Improve get thumbnail logic
a07e6d8
@djorkaeffalexandre djorkaeffalexandre marked this pull request as draft June 3, 2020 15:25
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 3, 2020

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

@diegolmello
Merge branch 'develop' into new.multiple-files
c318c8b 
# 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
@diegolmello
Header
57ec0ae
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 18, 2020

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
Copy link
Copy Markdown

lgtm-com bot commented Jun 25, 2020

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
Copy link
Copy Markdown

lgtm-com bot commented Jun 26, 2020

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

djorkaeffalexandre
djorkaeffalexandre commented Jun 26, 2020
View reviewed changes
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 26, 2020

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
Copy link
Copy Markdown

lgtm-com bot commented Jun 26, 2020

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
Copy link
Copy Markdown

lgtm-com bot commented Jun 26, 2020

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

@diegolmello diegolmello merged commit 07e9bcb into develop Jun 26, 2020
@diegolmello diegolmello deleted the new.multiple-files branch June 26, 2020 20:22
@ankar84 ankar84 mentioned this pull request Jun 29, 2020
Closed
diegolmello added a commit that referenced this pull request Jul 6, 2020
@diegolmello @phriedrich
@devyaniChoubey @Prateek93a @ssbing99 @djorkaeffalexandre @ura14h @Exordian @Daanchaam @youssef-md @ivanape @sarthakpranesh @tanmoyopenroot @hibikine @EzequielDeOliveira @regalstreak @dependabot @GOVINDDIXIT @azhaubassar @zhaubassar @Badal1234 @BrunoOliveiraDantas @CalebeRios @Fishbowler