Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

chore: update react-native from 0.60.x to 0.61.x #698

Merged
merged 6 commits into from
Jun 18, 2021
Merged

Conversation

patogallaiovlabs
Copy link
Collaborator

@patogallaiovlabs patogallaiovlabs commented Jun 10, 2021

Description

From previous step 0.59->0.60 (#690), this step goes from 0.60 to 0.61.
Changelog of 0.61: https://github.com/react-native-community/releases/blob/master/CHANGELOG.md#v0610
We followed this guide:

Common issues solved here:

How Has This Been Tested?

Doing some basics interactions, txs send, etc.

Types of changes

  • fix: bug fixes, e.g. fix Button color on DarkTheme.
  • feat: new features, e.g. add Snackbar component.
  • refactor: code refactor, e.g. new folder structure for components.
  • docs: changes into documentation, e.g. add usage example for Button.
  • test: adding or updating tests, eg unit, snapshot testing.
  • chore: tooling changes, e.g. change circleci config.
  • BREAKING CHANGE: for changes that break existing usage, e.g. change API of a component.

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • Tests for the changes have been added (for bug fixes / features)

NPM Audit

  • Before this PR:
    • found 239 vulnerabilities (26 low, 154 moderate, 59 high) in 2179 scanned packages
  • After this PR:
    • found 238 vulnerabilities (25 low, 154 moderate, 59 high) in 2197 scanned packages

@patogallaiovlabs patogallaiovlabs marked this pull request as ready for review June 11, 2021 19:34
@patogallaiovlabs patogallaiovlabs added this to the v2.0.0 milestone Jun 11, 2021
@patogallaiovlabs patogallaiovlabs self-assigned this Jun 11, 2021
Copy link
Contributor

@martinmedina martinmedina left a comment

Choose a reason for hiding this comment

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

The only comment made so far reflects what I found while checking this PR. I'll give it a final look once the comment is addressed.

android/app/src/main/AndroidManifest.xml Outdated Show resolved Hide resolved
@lucachaco
Copy link
Contributor

tRBTC sent from android
image

@lucachaco
Copy link
Contributor

tRBTC sent with IOS

image

Copy link
Contributor

@lucachaco lucachaco left a comment

Choose a reason for hiding this comment

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

LGTM. Works on ios and android

Copy link
Contributor

@martinmedina martinmedina left a comment

Choose a reason for hiding this comment

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

LGTM!

Base automatically changed from upgrade_deps to master June 18, 2021 20:59
@patogallaiovlabs patogallaiovlabs merged commit ed0b9b7 into master Jun 18, 2021
@patogallaiovlabs patogallaiovlabs deleted the rn_v0.61 branch June 18, 2021 21:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants