Skip to content

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