Skip to content

Convert to function component with hooks#8

Open
gmaclennan wants to merge 4 commits intoTitanInvest:masterfrom
gmaclennan:function-component
Open

Convert to function component with hooks#8
gmaclennan wants to merge 4 commits intoTitanInvest:masterfrom
gmaclennan:function-component

Conversation

@gmaclennan
Copy link
Copy Markdown

I updated this to use a function component, mainly for my own use, but leaving this here in case you want to want to update this library. This removes the deprecated componentWillReceiveProps. It depends on hooks, so will only work with react-native@^0.59.0. Also removes babel and compile step, since it's not needed for react-native use.

@Danny-A
Copy link
Copy Markdown

Danny-A commented May 21, 2020

Would be nice if this could be updated @zivester

@edreyyo
Copy link
Copy Markdown

edreyyo commented Sep 25, 2020

Any chance this could get merged, @zivester? 🙏

@alexpchin
Copy link
Copy Markdown

@zivester Would it be possible to merge this please?

@zivester zivester requested review from Crizzooo and zivester October 27, 2020 19:52
@Crizzooo
Copy link
Copy Markdown

Thanks for the conversion to a function component, Im sorry we didnt notice this for months.

This package is a bit tricky and is used in a few of our other apps, will test the new build process and update later this week.

@edreyyo
Copy link
Copy Markdown

edreyyo commented Dec 7, 2020

Could someone please resolve these conflicts and merge the PR, @zivester @Crizzooo? (Assuming everything else is OK)

@alexpchin
Copy link
Copy Markdown

@Crizzooo @zivester Any chance of getting this merged? Thanks!

@zivester zivester removed their request for review September 30, 2021 14:12
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.

5 participants