-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue/12597 custom fields banner #12601
base: trunk
Are you sure you want to change the base?
Conversation
private val appPrefs: AppPrefsWrapper = mock { | ||
val bannerDismissed = MutableStateFlow(false) | ||
on { observePrefs() }.thenReturn(bannerDismissed.map { Unit }) | ||
on { isCustomFieldsTopBannerDismissed } doAnswer { bannerDismissed.value } | ||
on { isCustomFieldsTopBannerDismissed = any() }.then { invocation -> | ||
bannerDismissed.update { invocation.arguments[0] as Boolean } | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works more like a fake than a mock, and allows for a more thorough testing (we can simulate the dismiss properly).
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
89982ce
to
bb72363
Compare
Closes: #12597
Description
This PR adds a banner to the custom fields list screen to show a message that would help explaining how the saving works for them, in contrast to how it works in the other parts of the Product Details.
For more information on why we need this and how we reached this decision please check this discussion p1725968896022239/1725948493.332539-slack-C03L1NF1EA3
Steps to reproduce
Testing information
The tests that have been performed
The above.
Images/gif
RELEASE-NOTES.txt
if necessary. Use the "[Internal]" label for non-user-facing changes.Reviewer (or Author, in the case of optional code reviews):
Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement: