Skip to content

Conversation

@AdamGrzybkowski
Copy link
Contributor

Description

Follow-up to #14660

More context: p1759140058893509-slack-C03L1NF1EA3

This PR updates the icons, the copy for the Cancelled state and the order in the bottom sheet.

Steps to reproduce

  1. Log in.
  2. Go to Menu → Settings → Developer Options → Show booking details
  3. Scroll down to the attendance section
  4. Verify the UI

Testing information

The above steps should be enough.

The tests that have been performed

Images/gif

Screenshot_20250929_135137
  • I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

@AdamGrzybkowski AdamGrzybkowski added this to the 23.4 milestone Sep 29, 2025
@AdamGrzybkowski AdamGrzybkowski changed the title [WOOMOB-1345] Follow-up to the previous PR with updated ico [WOOMOB-1345] Booking attendance bottom sheet icons Sep 29, 2025
@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commita4201e9
Direct Downloadwoocommerce-wear-prototype-build-pr14662-a4201e9.apk

@wpmobilebot
Copy link
Collaborator

📲 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
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commita4201e9
Direct Downloadwoocommerce-prototype-build-pr14662-a4201e9.apk

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 38.40%. Comparing base (5beb0bb) to head (a4201e9).

Files with missing lines Patch % Lines
...ings/compose/BookingAttendanceStatusBottomSheet.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##              trunk   #14662   +/-   ##
=========================================
  Coverage     38.39%   38.40%           
- Complexity     9799     9801    +2     
=========================================
  Files          2083     2083           
  Lines        116273   116273           
  Branches      15534    15534           
=========================================
+ Hits          44646    44650    +4     
+ Misses        67479    67477    -2     
+ Partials       4148     4146    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@irfano irfano self-assigned this Sep 29, 2025
Copy link
Contributor

@irfano irfano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍🏻

@irfano irfano merged commit ac2e5e7 into trunk Sep 29, 2025
18 of 21 checks passed
@irfano irfano deleted the issue/WOOMOB-1345_attendance_section_icons branch September 29, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants