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

Upgrade react #862

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Upgrade react #862

wants to merge 3 commits into from

Conversation

Arjun-sna
Copy link
Contributor

Question Response
Version? v1.4.1
Devices tested? oneplus 5, iphone SE
Bug fix? no
New feature? no
Includes tests? no
All Tests pass? yes
Related ticket? #859

Screenshots

Description

Upgrade react version to 16.6.0
Refactored old react lifecycle functions with new functions

@coveralls
Copy link

coveralls commented Jan 8, 2019

Coverage Status

Coverage remained the same at 48.608% when pulling 8a733fe on Arjun-sna:upgrade_react into 41b3a72 on gitpoint:master.

@Arjun-sna Arjun-sna mentioned this pull request Feb 1, 2019
@chinesedfan
Copy link
Member

I upgraded React/ReactNative in #883. Would you mind refactoring old react lifecycle functions again? Or we can record files as a new issue and refactor one by one.

@Arjun-sna
Copy link
Contributor Author

@chinesedfan Please create a new issue with a list of files need to be updated. I will try to work on those as much as possible. Thanks.

@chinesedfan
Copy link
Member

@Arjun-sna See #887. You have did most of them I think.

@Arjun-sna
Copy link
Contributor Author

@chinesedfan Sorry for the delayed update.
I have updated the PR with required changes for latest react.

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