Skip to content

fix: best-effort dismiss of Background Payments intro sheet - #242

Open
jvsena42 wants to merge 2 commits into
mainfrom
fix/901-block-own-invoice
Open

jvsena42 wants to merge 2 commits into
mainfrom
fix/901-block-own-invoice

Conversation

@jvsena42

Copy link
Copy Markdown
Member

Companion branch for synonymdev/bitkit-android#1292 (fix/901-block-own-invoice), so the local e2e job resolves a matching branch here.

Same commit as the other companions on this batch (#234, #236, #237, #238, #239, #240, #241): dismissBackgroundPaymentsTimedSheet polls for the sheet and swallows the timeout when it never appears, instead of failing the spec.

Without it, the settings shard on #1292 fails @settings_01 and @settings_06 with DrawerSettings still not displayed after 30000ms — the intro sheet sits over the home screen and swallows the drawer tap. The Android PR itself only adds a self-payment guard in the send flow and does not touch settings or the drawer.

Hard-waiting on BackgroundPaymentsIntro-later flakes the Android send shard when the timed sheet is already gone or auto-dismissed. Poll like waitForToastBestEffort and still dismiss Later/Cancel when the intro is shown.

Co-authored-by: piotr-iohk <piotr-iohk@users.noreply.github.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 9b95a97)
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.

2 participants