transferZ icon indicating copy to clipboard operation
transferZ copied to clipboard

Flutter [Null Safety Migration]

Open abinvs-2019 opened this issue 3 years ago • 6 comments

abinvs-2019 avatar Aug 26 '22 11:08 abinvs-2019

There is error while building the app from the kotlin side. That should also be updated.

abinvs-2019 avatar Aug 26 '22 11:08 abinvs-2019

I see too many file changes. It'll be little hard to review.

itzmeanjan avatar Aug 27 '22 08:08 itzmeanjan

Yes there is too many changes because of migration to null safety. Also the app has issue form kotlin side. That should be taken care. Please do review and merge this request to move forward

abinvs-2019 avatar Sep 21 '22 08:09 abinvs-2019

I see too many file changes. It'll be little hard to review.

@itzmeanjan Is there anu update on the PR. I think there are no conflicts on the base branch. if there is no issues could you merge it. So we could fix other issues.

abinvs-2019 avatar Sep 26 '22 11:09 abinvs-2019

I see too many file changes. It'll be little hard to review.

@itzmeanjan Is there anu update on the PR. I think there are no conflicts on the base branch. if there is no issues could you merge it. So we could fix other issues.

Hi, I've not got the opportunity to review the changes yet. Also I'm not maintaining this project for sometime now, so it might be little hard for me to review and get it merged.

itzmeanjan avatar Sep 26 '22 17:09 itzmeanjan

So should I close the PR.?

abinvs-2019 avatar Oct 15 '22 10:10 abinvs-2019

Hi, thanks for the PR, but I've to archive this repository.

itzmeanjan avatar Dec 17 '22 09:12 itzmeanjan