-
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/12563 delete custom fields #12566
Conversation
📲 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.
|
This allows having the Snackbar on the correct location above the FAB, instead of hiding it.
c20d410
to
15dbfa7
Compare
...e/src/main/kotlin/com/woocommerce/android/ui/customfields/editor/CustomFieldsEditorScreen.kt
Show resolved
Hide resolved
...ommerce/src/main/kotlin/com/woocommerce/android/ui/customfields/list/CustomFieldsFragment.kt
Show resolved
Hide resolved
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## trunk #12566 +/- ##
============================================
+ Coverage 40.60% 40.61% +0.01%
- Complexity 5677 5680 +3
============================================
Files 1228 1228
Lines 69062 69091 +29
Branches 9568 9572 +4
============================================
+ Hits 28044 28064 +20
- Misses 38437 38444 +7
- Partials 2581 2583 +2 ☔ View full report in Codecov by Sentry. |
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.
LGTM! 👍🏻 Works well! I added a comment and will approve and merge after you respond to it.
...src/main/kotlin/com/woocommerce/android/ui/customfields/editor/CustomFieldsEditorFragment.kt
Show resolved
Hide resolved
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.
Good job! LGTM! You'll probably do, but as a reminder, don't forget to add RELEASE NOTES in one of your PRs for the custom fields feature.
Thanks @irfano for the reminder, what we do generally when working on a feature that's behind a feature flag, is that we add the release notes entry in the PR that enables or removes the feature flag, so that it's not announced before we release it. |
Closes: #12563
Description
This PR adds support for deleting custom fields.
Steps to reproduce
Testing information
The tests that have been performed
Same as 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: