nativescript-contacts icon indicating copy to clipboard operation
nativescript-contacts copied to clipboard

Published 20 hours ago • verified publisher icon firescript

Bug when clicking on cancel with version 1.6.3

Open Bidou44 opened this issue 3 years ago • 7 comments

After updating to version 1.6.3 and releasing on prod, I can confirm that the bug https://github.com/firescript/nativescript-contacts/issues/75 is now fixed! Thanks again @dlcole @firescript

Unfortunately, I also noticed that a new issue raised up. Looking at the statistics it seems that 1-2% of the users have an issue that makes the application crash. Actually it's quite simple to reproduce, you have to open the contact plugin and click on cancel to close it. Usually it's working, but sometime it's not. I was able to reproduce it on some phone and some other not.

On average, I would say that I have to open/close the contact plugin more or less 5 times before the application crashes (sometimes it crashes after the first time, sometimes you have to try 20 times and as said, on some devices, its never crashing...)

Any idea what the problem could be? In my opinion, it's new with the version 1.6.3

Bidou44 avatar Jun 16 '21 05:06 Bidou44

After a few days of use on production, the statistics of the users having a crash increased to more than 2% (only iOS). I had to do a rollback. I cannot recommend using the latest version 1.6.3 (even if it solves an issue) as it is more buggy that the previous, unfortunately.

Bidou44 avatar Jun 19 '21 05:06 Bidou44

@firescript Further info: Doing a rollback didn't help (App is always crashing for 2% if my user on prod) so I think that this bug is caused by Nativescript 8 (while moving to nativescript-contacts 1.6.3 I also upgraded to Nativescript 8). Maybe it's worth looking at this PR: https://github.com/firescript/nativescript-contacts/pull/97 What do you think?

Bidou44 avatar Jun 21 '21 10:06 Bidou44

We could merge that, and bump npm again. Would that solve your issue?

firescript avatar Jun 21 '21 12:06 firescript

I didn't check the content/quality of this PR, but if the migration was correctly done, then yes if would really help!

Bidou44 avatar Jun 21 '21 13:06 Bidou44

@firescript Any news on this? Would it be possible to merge the PR in a branch and publish a beta version? I can then do some tests to check if is solves the problem.

Bidou44 avatar Jun 24 '21 09:06 Bidou44

Now that the pull request 97 is merged , waiting for the next release. Is it planned Ryan?

raphaelguye avatar Jul 14 '21 05:07 raphaelguye

@firescript Yes it would be good if you could release a new version at least a beta so that I can make some tests...

Bidou44 avatar Jul 15 '21 12:07 Bidou44