bitkit icon indicating copy to clipboard operation
bitkit copied to clipboard

[Bug]: Paying to Contacts functionality broken

Open aldertnl opened this issue 3 years ago • 6 comments

Describe the bug

The 'Pay to Contacts' functionality seems to be broken in v1.0 (21) and v1.0 (22). A user cannot select a contact from the contact list in the send flow. Also, tapping the [coins] button while viewing a contact profile will not activate the 'send to contact' modal.

This is an important issue to fix in my opinion, as sending to contacts is one of the most important use cases within Bitkit.

Reproduce

  1. Go to [Send]

  2. Click on [Contacts icon]

  3. Select a contact from the contacts list

  4. Nothing happens

  5. Go to [Contacts]

  6. Tap on a contact

  7. Tap on the [coins] button

  8. Nothing happens

Screenshots / Recording

No response

Operating system

iPhone 11, iOS 16.1.1

Bitkit version

1.0 (22)

Log output

No response

aldertnl avatar Nov 29 '22 15:11 aldertnl

image

I ran into this issue as well

Bitc0indad avatar Dec 01 '22 15:12 Bitc0indad

This is a different issue (but related). Some team members encountered the same error as you. @Nazeh @BitcoinErrorLog

aldertnl avatar Dec 01 '22 18:12 aldertnl

This may be the same issue, as at first I thought I also only could not tap on any contact, but then I realized this error started showing and that was why.

This could be because have bad handling of the fallback, a missing fallback, or failed replacement processes. Needs more attention and testing of fixes.

BitcoinErrorLog avatar Dec 01 '22 18:12 BitcoinErrorLog

This is either slashpay.json wasn't resolved (seeder didn't have the data, and the user is offline), or the user didn't allow slashpay at all... either way there is no way to tell which.

However we can add an error message if that was the case, so it is a bit distinct from this generic error that is thrown at the end of the code path (that is common with scanning normal bitcoin/lightning QR)

Nuhvi avatar Dec 02 '22 14:12 Nuhvi

It is probably something broad, as it happens for all my contacts the same way. Something is broken. Please test using the app in two phones across the team, and with team members that are having the issue. Also test while app is closed vs open, etc. Generally Slashtags Profiles, Contacts, and Slashpay are not reliable yet.

BitcoinErrorLog avatar Dec 02 '22 14:12 BitcoinErrorLog

Tested this feature again today with @pwltr, we had varying experiences (v1.0 (23)).

  • I couldn't select any of the contacts in my contacts list (mostly team members), except for a newly generated profile by Philipp and a test profile.
  • Philip could select most contacts in his contact list, and it would default to the on-chain Pay to Contacts flow (expected behavior).
  • Using his new profile, I was able to successfully send an on-chain transaction to Philipp using the Pay to Contacts flow.

Some current presumptions based on these and previous observations:

  • This issue seems to affect 'older' contacts more than recently created ones.
  • This issue seems to mostly affect users that simply updating the app instead of deleting, reinstalling and restoring their wallet.

We couldn't test the LN instant payment variant of the Pay to Contacts flow, as Philipp had an issue with setting up a new instant spending balance (looks like a Blocktank issue).

aldertnl avatar Dec 05 '22 18:12 aldertnl

Update: The Pay to Contacts feature is still not working for me 90+ % of the time (can't be the only one). Getting a spinner for almost every contact that I select from the contact list (Send Flow, build v31). Can someone confirm this?

5F9AC607-E6A7-4EBB-AA11-EB4A580D9993_1_102_o

aldertnl avatar Jan 30 '23 17:01 aldertnl

@aldertnl the internal bulid is disabling the Hyperswarm connection to the seeder, I am trying to add a stopgap WebSocket connection to that seeder, and it should be there in the next internal build.

Nuhvi avatar Jan 30 '23 18:01 Nuhvi

Did this change make it into our new internal build 33? I am not seeing any improvements, still getting the spinner when selecting a contact.

aldertnl avatar Feb 02 '23 23:02 aldertnl

@aldertnl we will see in today's internal build, it should work nicely fingers crossed #870

Nuhvi avatar Feb 03 '23 07:02 Nuhvi

should we close this issue already?

Nuhvi avatar Feb 07 '23 15:02 Nuhvi

Please test after the next release with Pav/Jan and confirm all test cases work (new user payments, onchain/offchain, replacement invoices)

BitcoinErrorLog avatar Feb 07 '23 15:02 BitcoinErrorLog

https://synonymworkspace.slack.com/archives/C01H106JRQB/p1676540215532869?thread_ts=1676526564.246289&cid=C01H106JRQB

Nuhvi avatar Feb 16 '23 09:02 Nuhvi