Skip to content

Conversation

@diegolmello
Copy link
Member

@diegolmello diegolmello commented Jul 4, 2023

Proposed changes

Depends on RocketChat/rn-extensions-share#10

Issue(s)

How to test or reproduce

  • Test Android drawer performance
  • Test as beta

Screenshots

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • 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

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • 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

NATIVE-368

@diegolmello diegolmello force-pushed the chore.upgrade-rn72 branch from f8fb24a to 71dafd3 Compare July 5, 2023 16:27
@diegolmello diegolmello force-pushed the chore.upgrade-rn72 branch from 942bab4 to 72fa2ea Compare July 5, 2023 18:06
@diegolmello diegolmello force-pushed the chore.upgrade-rn72 branch from 44189f4 to c5726db Compare July 5, 2023 21:55
@diegolmello diegolmello force-pushed the chore.upgrade-rn72 branch from c5726db to 1d99145 Compare July 6, 2023 13:12
- restore_cache: *restore-gradle-cache

- save_cache: *save-npm-cache-linux
# - restore_cache: *restore-gradle-cache
Copy link
Member Author

@diegolmello diegolmello Jul 17, 2023

Choose a reason for hiding this comment

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

Had to do it to build properly.
Removed here only, because e2e tests seems to work with it.
Probably because here it's bundling, while there it's assembling.
After we get rid of the RN Android fork, we might be able to enable it again.

@diegolmello diegolmello marked this pull request as ready for review July 17, 2023 14:32
@diegolmello diegolmello changed the title feat: Upgrade to React Native 0.72.1 feat: Upgrade to React Native 0.72.3 Jul 17, 2023
Copy link
Contributor

@dnlsilva dnlsilva left a comment

Choose a reason for hiding this comment

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

did checkout on branch .
deleted the node_modules folder and yarn again.
pod install (Here there was an error, but running pod update worked)
When clicking on the emoji in any conversation, this error appears:
Simulator Screen Shot - iPhone 14 - 2023-07-17 at 13 59 21

diegolmello and others added 2 commits July 17, 2023 14:06
Co-authored-by: Gleidson Daniel Silva <[email protected]>
Co-authored-by: Gleidson Daniel Silva <[email protected]>
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.

3 participants