Skip to content

[CHORE] Migrate to Watermelon#1171

Merged
diegolmello merged 226 commits intodevelopfrom
watermelon-poc
Sep 16, 2019
Merged

[CHORE] Migrate to Watermelon#1171
diegolmello merged 226 commits intodevelopfrom
watermelon-poc

Conversation

@diegolmello
Copy link
Copy Markdown
Member

@diegolmello diegolmello commented Sep 2, 2019

@RocketChat/ReactNative

  • Configure (iOS and Android)
  • Load rooms
  • Load room
  • Update rooms (socket)
  • Sync rooms
  • Update room (socket)
  • Sync room
  • Load more messages for room
  • Multiple servers
  • Share extension
  • Message actions (reply, edit, pin, star, etc)
  • Error actions (try again, delete message)
  • Emoji picker
  • User status (active users)
  • Permissions
  • Settings
  • Roles
  • Slash commands
  • Threads
  • Discussions
  • Uploads
  • User typing
  • Clear database on logout
  • Fav, hide, mark as unread room
  • Sort rooms
  • Group rooms
  • Search rooms
  • Save user
  • Unread count on back button
  • Save Notifications
  • Receive Notifications
  • Deep linking
  • Edit room
  • Draft
  • Work offline
  • Muted
  • NewMessage
  • Add user
  • Create channel
  • Test OAuth
  • Test sync credentials with iOS native
  • Performance
  • E2E tests
  • Remove Realm completely
  • Test again (:

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Sep 13, 2019

This pull request introduces 6 alerts when merging df281d1 into 2d8b1c5 - view on LGTM.com

new alerts:

  • 3 for Direct state mutation
  • 2 for Useless conditional
  • 1 for Unsupported state update in lifecycle method

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Sep 13, 2019

This pull request introduces 6 alerts when merging 4315910 into 2d8b1c5 - view on LGTM.com

new alerts:

  • 3 for Direct state mutation
  • 2 for Useless conditional
  • 1 for Unsupported state update in lifecycle method

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Sep 16, 2019

This pull request introduces 11 alerts when merging 97703ac into 2d8b1c5 - view on LGTM.com

new alerts:

  • 8 for Direct state mutation
  • 2 for Useless conditional
  • 1 for Unsupported state update in lifecycle method

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Sep 16, 2019

This pull request introduces 11 alerts when merging d2a5232 into 2d8b1c5 - view on LGTM.com

new alerts:

  • 8 for Direct state mutation
  • 2 for Useless conditional
  • 1 for Unsupported state update in lifecycle method

@diegolmello diegolmello changed the title Migrate to Watermelon [CHORE] Migrate to Watermelon Sep 16, 2019
Copy link
Copy Markdown
Contributor

@djorkaeffalexandre djorkaeffalexandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@diegolmello diegolmello merged commit 9ba3710 into develop Sep 16, 2019
@diegolmello diegolmello deleted the watermelon-poc branch September 16, 2019 20:26
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Sep 16, 2019

This pull request introduces 11 alerts when merging 65c5185 into 2d8b1c5 - view on LGTM.com

new alerts:

  • 8 for Direct state mutation
  • 2 for Useless conditional
  • 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.

3 participants