woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
262 stars 133 forks source link

[Dynamic Dashboard] Beta release fixes #11526

Closed 0nko closed 2 weeks ago

0nko commented 2 weeks ago

Fixes #11523 and addresses the issues found during beta testing.

To test:

  1. Verify the dashboard screen menu buttons match the design (first share button
  2. Verify the custom date range pencil icon matches the design (same as dashboard Edit menu button)
  3. Verify that after trashing the only order, the dashboard cards related to orders are hidden
dangermattic commented 2 weeks ago
1 Warning
:warning: This PR is assigned to the milestone 18.7. This milestone is due in less than 2 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by :no_entry_sign: Danger

wpmobilebot commented 2 weeks ago

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

App Name WooCommerce Android
FlavorJalapeno
Build TypeDebug
Commit3ea8b119ff03e0e038ecc4f9f31989f1c0345b52
Direct Downloadwoocommerce-prototype-build-pr11526-3ea8b11.apk
codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 40.45%. Comparing base (293506d) to head (3ea8b11).

Files Patch % Lines
...android/ui/dashboard/stats/DashboardStatsHeader.kt 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #11526 +/- ## ============================================ - Coverage 40.45% 40.45% -0.01% Complexity 5181 5181 ============================================ Files 1082 1082 Lines 62904 62905 +1 Branches 8616 8616 ============================================ Hits 25447 25447 - Misses 35169 35170 +1 Partials 2288 2288 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.