Skip to content

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

Closed
jvsena42 wants to merge 2 commits into
mainfrom
fix/channel-details-time-format
Closed

jvsena42 wants to merge 2 commits into
mainfrom
fix/channel-details-time-format

Conversation

@jvsena42

Copy link
Copy Markdown
Member

Companion branch for synonymdev/bitkit-android#1123 (fix/channel-details-time-format), so the local e2e job resolves a matching branch here instead of falling back to main.

Same commit as the other companions on this batch: dismissBackgroundPaymentsTimedSheet polls for the sheet and swallows the timeout when it never appears, rather than failing the spec. Without it the Background Payments intro sits over Home and swallows the drawer tap, which surfaces as DrawerSettings still not displayed after 30000ms in the settings shard.

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 jvsena42 self-assigned this Sep 18, 2026
@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/channel-details-time-format 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