Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

React Native update to 0.63 #2078

Merged
merged 32 commits into from
Feb 4, 2021
Merged

Conversation

andrepimenta
Copy link
Member

@andrepimenta andrepimenta commented Dec 16, 2020

Description

This updates React Native from 0.62.2 to 0.63.4:
https://react-native-community.github.io/upgrade-helper/?from=0.62.2&to=0.63.4

@ibrahimtaveras00 Important to note that:

  • react navigation was updated
  • I noticed some text inputs changed, like font size, I fixed the ones I found but may exist more
  • react-native-keyboard-aware-scroll-view was updated, this exists mainly on scrollable screens or components

Issue

Resolves #2065

@andrepimenta andrepimenta requested a review from a team as a code owner December 16, 2020 16:54
@andrepimenta andrepimenta added needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) needs-qa Any New Features that needs a full manual QA prior to being added to a release. labels Dec 16, 2020
@andrepimenta andrepimenta changed the title Improvement/react native update 0.63 React Native update to 0.63 Dec 16, 2020
Copy link
Member

@wachunei wachunei left a comment

Choose a reason for hiding this comment

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

code looks good to me, but I'd definitely be very cautious about all the app's text inputs if they changed the default style.

@omnat omnat added design-review Any feature that needs feedback from the design team next release needs-design Feature that requires UI/UX design labels Jan 5, 2021
@estebanmino estebanmino removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Jan 5, 2021
@omnat omnat added the DO-NOT-MERGE Pull requests that should not be merged label Jan 6, 2021
Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

Issue 1:

When on iOS and going through the onboarding wizard, when I'm all the way on browser, and go back to the wallet, the wizard disappears and I'm stuck until I tap skip tutorial

seen here = https://recordit.co/8r4u7d2j9h

Issue 2:

When you're on android and you are backing up seed phrase, when you tap and then deselect the pills they remove the text and is no longer legible

seen here:
image

@ibrahimtaveras00 ibrahimtaveras00 added QA'd - Issues Found QA has been complete, however issues have been discovered that need to be addressed and removed needs-qa Any New Features that needs a full manual QA prior to being added to a release. labels Jan 12, 2021
@andrepimenta andrepimenta added needs-qa Any New Features that needs a full manual QA prior to being added to a release. and removed QA'd - Issues Found QA has been complete, however issues have been discovered that need to be addressed labels Jan 19, 2021
# Conflicts:
#	ios/Podfile.lock
#	package.json
@andrepimenta andrepimenta removed the DO-NOT-MERGE Pull requests that should not be merged label Feb 1, 2021
Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

QA Passed 👍🏽

@ibrahimtaveras00 ibrahimtaveras00 added QA Passed A successful QA run through has been done and removed design-review Any feature that needs feedback from the design team needs-design Feature that requires UI/UX design needs-qa Any New Features that needs a full manual QA prior to being added to a release. labels Feb 3, 2021
# Conflicts:
#	android/app/build.gradle
#	ios/MetaMask.xcodeproj/project.pbxproj
#	ios/Podfile.lock
#	package.json
#	yarn.lock
@andrepimenta andrepimenta merged commit 8b74b64 into develop Feb 4, 2021
@andrepimenta andrepimenta deleted the improvement/react-native-update-0.63 branch February 4, 2021 15:14
rickycodes added a commit that referenced this pull request Feb 25, 2021
* develop: (48 commits)
  Fix input state (#2265)
  bump android versionCode (#2260)
  bugfix/use bignumber for transfer deeplinks (#2257)
  Fix account list scroll (#2256)
  TransactionController from controllers (#2243)
  Fix circleci apk (#2240)
  v1.0.11 (#2238)
  Feature/warn when replacing (#2176)
  contract metadata images bump (#2233)
  Handle `balanceError` case (#2148)
  Remove best deals badge from WYRE transfers. (#2228)
  @metamask/controllers@6.0.1 (#2217)
  Fix cloudflare redirects (#2180)
  Merge pull request from GHSA-wmvx-96jh-gcr4
  Update @metamask/contract-metadata (#2203)
  React Native update to 0.63 (#2078)
  Create dependabot.yml (#2204)
  Fix "Text strings must be rendered within a <Text> component" (#2193)
  Use navigate instead of push (#2191)
  Add fiat on ramp modal close button extra hit area (#2174)
  ...
rickycodes pushed a commit that referenced this pull request Jan 31, 2022
* Files modified

* Yellow box to logbox and keyboard aware and eslint lib update

* Update react navigation and fix inputs size

* Update scrollview tests

* keyring unit test

* More scroll view unit testing update

* Fix typo

* Updating password unit tests

* snaps

* circle

* 9

* CURRENT_PROJECT_VERSION

* xcodeversioncircleci

* gesture-handler

* lock

* xcodeversioncircleci

* xcodeversioncircleci

* xcodeagain

* intallbundler

* faraday

* v

* fastlane

* bumps

* ifdebug

* Fix detox for android and ios builds

* Fix tutorial on iOS and backup on android

* Update yarn.lock

* Remove ignore from ci

* Update unit tests

Co-authored-by: Esteban Mino <efmino@uc.cl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release QA Passed A successful QA run through has been done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update React Native to 0.63
5 participants