Skip to content

Fix contacts permission prompt timing#5979

Open
j0ntz wants to merge 3 commits intodevelopfrom
jon/fix/remove-contacts-modal-first-receive
Open

Fix contacts permission prompt timing#5979
j0ntz wants to merge 3 commits intodevelopfrom
jon/fix/remove-contacts-modal-first-receive

Conversation

@j0ntz
Copy link
Contributor

@j0ntz j0ntz commented Mar 13, 2026

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Description

Asana task

Remove contacts modal on first receive. The contacts permission prompt no longer appears from passive thumbnail lookup when a user receives funds. It now appears only when the user opens a transaction list with named transactions or explicitly edits the payee field from transaction details.

j0ntz added 2 commits March 13, 2026 11:48
Introduce a non-deprecated flat navigation alias for shared code and update the transaction list scene to use it for legacy cross-navigator casts.
Clean up the existing contacts permission modal warnings so the feature work can build on a warning-free file.
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Prevent contact permission prompts from appearing during passive thumbnail lookups and limit them to transaction-list or explicit payee edit flows.
@j0ntz j0ntz force-pushed the jon/fix/remove-contacts-modal-first-receive branch from b672ee4 to 8cda7cb Compare March 13, 2026 22:38
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.

1 participant