Skip to content

Conversation

@RafaelKayumov
Copy link
Contributor

@RafaelKayumov RafaelKayumov commented Oct 28, 2025

WOOMOB-1532

Description

As suggested here - adding an attendance status update failure notice with retry CTA.

  • Adds a notice: Notice? to view model
  • Presents the view model notice in BookingDetailsView
  • The notice calls the updateAttendanceStatus(to: _) on a retry CTA
  • Adds tests

Testing Steps

  • Use a CIAB site
  • Find a non-cancelled booking and open booking details
  • Scroll down to "Attendance" section.
  • Turn off internet or make a breakpoint with Proxyman for the /wc-bookings/v2/bookings/{booking_id}&_method=put
  • Tap on the attendance status row and select a new status
  • Make sure the request is failed by emulating 500 error in Proxyman or by keeping the interned disabled.
  • Make sure the failure notice is displayed
  • Disable the breakpoint / turn on the internet and tap on the retry (make it quick :))
  • Make sure the status change is retried and re-applied properly.

Screenshots

Снимок экрана 2025-10-28 в 18 23 38
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@RafaelKayumov RafaelKayumov added this to the 23.6 milestone Oct 28, 2025
@RafaelKayumov RafaelKayumov added the type: task An internally driven task. label Oct 28, 2025
@wpmobilebot
Copy link
Collaborator

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

App NameWooCommerce iOS Prototype
Build Numberpr16286-3a69851
Version23.5
Bundle IDcom.automattic.alpha.woocommerce
Commit3a69851
Installation URL4nhgsn5uu3440
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@itsmeichigo itsmeichigo self-assigned this Oct 29, 2025
Copy link
Contributor

@itsmeichigo itsmeichigo left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@RafaelKayumov RafaelKayumov merged commit d44e9b5 into trunk Oct 29, 2025
17 of 19 checks passed
@RafaelKayumov RafaelKayumov deleted the WOOMOB-1553-status-update-failure-notice-and-retry branch October 29, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants