-
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
Reorder cards #10937
Reorder cards #10937
Conversation
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## issue/10702-hide-cards-based-on-configuration #10937 +/- ##
===================================================================================
- Coverage 41.34% 41.27% -0.08%
- Complexity 5054 5055 +1
===================================================================================
Files 1028 1030 +2
Lines 58860 58980 +120
Branches 7854 7878 +24
===================================================================================
+ Hits 24337 24342 +5
- Misses 32357 32470 +113
- Partials 2166 2168 +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.
Amazing improvement, @atorresveiga 👏 The Analytics Hub is becoming such an impressive feature 🥲 Looks good!
…ue/10703-reorder-cards
Part of: #10703
Why
As part of Expanded Analytics Hub milestone 2, we plan to add support for customizing the cards we display on the Analytic Hub screen.
Description
This PR adds support for reordering the analytics cards. It does that by adding drag-and-drop support for the compose lazy list and updating the order of cards on the ViewModel. Because we already were persisting the list on the device, not further changes were required.
Testing instructions
TC1
Images/gif
Screen_recording_20240227_115839.mp4
RELEASE-NOTES.txt
if necessary. Use the "[Internal]" label for non-user-facing changes.