Conversation
|
Regtest APKDownload bitkit-dev-debug universal APK (expires in 30 days). |
jvsena42
left a comment
There was a problem hiding this comment.
No findings beyond the open greptile thread on the dev-mode toast flash.
Checked and clean:
- The toggle only appears in Dev Settings, and the mask is
isDevModeEnabled && disableAllToasts, so it is off in release until dev mode is enabled, and turning dev mode off restores toasts. - It defaults to off and persists via
SettingsStore.update. ToastOverlayinMainActivityis the single chokepoint and reads the maskedcurrentToast, so everyToastEventBus/app.toastpath is covered.- The queue keeps running while hidden, and the only
autoHide = falsetoasts are always replaced, so nothing stale reappears. AppViewModelSendFlowTestasserts on, dev-off and toggle-off against the real flow.
Parity with synonymdev/bitkit-ios#829: same semantics and tag. iOS filters when a toast is requested, Android masks at display time.
Closes #1369
Twin: synonymdev/bitkit-ios#829
This PR adds a Dev Settings toggle that hides all toasts, for testing and demo recordings.
Description
Out of Scope
Design
N/A — no design available.
Preview
QA Notes
Journeys
temporary
disable-all-toasts.xml— toasts stay hidden while dev mode and the toggle are on, and return once dev mode is offdisable-all-toasts.xml
Manual Tests
N/A
Automated Checks
AppViewModelSendFlowTest.kt— the current toast is hidden only while dev mode and the toggle are both onSettingsViewModelTest.kt— setting dev mode completes only after the settings write finishes