Skip to content

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

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

jvsena42 wants to merge 3 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>
jvsena42 and others added 2 commits September 18, 2026 15:01
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 9b95a97)
@piotr-iohk

Copy link
Copy Markdown
Collaborator

Closing this empty same-branch companion.

It had 0 file changes vs main, but CI still checked out the stale tip (behind main). That tip still had the flaky Android NavigationBack tap after transfer that e2e#225 removed on main (~12:43 CEST today). Sibling Android PRs that already fell through to main stayed ✅ on lnurl_transfer / @transfer_2.

Closing + deleting the branch so determine-e2e-branch falls through to main. No harness change intended — app PR E2E will be fully re-run from determine.

@piotr-iohk

Copy link
Copy Markdown
Collaborator

Closing empty companion; see above.

@piotr-iohk piotr-iohk closed this Sep 21, 2026
@piotr-iohk
piotr-iohk deleted the fix/901-block-own-invoice branch September 21, 2026 12:55
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.

3 participants