Skip to content

Conversation

@RafaelKayumov
Copy link
Contributor

@RafaelKayumov RafaelKayumov commented Oct 31, 2025

WOOMOB-1616

Description

Implements navigation to a booking order from "Booking Details" screen

  • Re-uses MainTabBarController.navigateToOrderDetails(with: , siteID: ) for navigation
  • Opens the order in "Orders" tab
  • Applies the navigation for "View Order" CTA button and ellipsis action.
  • The missing order error fallback is already handled by Order Details screen.

Test Steps

  • Use CIAB sites with existing bookings and corresponding orders
  • Navigate to a Booking that has an order (Some booking don't)
  • Find and tap "View Order" button in Booking Details screen.
  • Make sure the app navigates to "Orders" tab and opens/loads the right order for that booking
  • Go back to Booking Details and tap on ... (ellipsis) button in top rigt. Select "View Order"
  • Make sure the right order is opened again in Orders tab.
  • Find another booking with existing order and repeat steps to double check
  • Find a Booking with missing order.
  • Trigger the "View Order" in booking details and make sure the "The Order couldn't be loaded!" + "Retry" error state is displayed in Order details screen.

Screenshots

NavigateToOrderDemo.mov

  • 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.7 milestone Oct 31, 2025
@RafaelKayumov RafaelKayumov added the type: task An internally driven task. label Oct 31, 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 Numberpr16303-9757fb5
Version23.6
Bundle IDcom.automattic.alpha.woocommerce
Commit9757fb5
Installation URL63vfu700ghnq0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@itsmeichigo itsmeichigo self-assigned this Nov 3, 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.

Code looks good and works as described 👍

I found an edge case: if you open the order for a booking then navigate back and try again, the order detail page will not be opened again on iPhone. I suppose this is an issue with MainTabBarController.navigateToOrderDetails, so pre-approving - please feel free to log a separate issue if this is tricky to fix.

Simulator.Screen.Recording.-.iPhone.17.-.2025-11-03.at.11.14.04.mov

@RafaelKayumov
Copy link
Contributor Author

Thx. I'll address that as a separate issue: WOOMOB-1653

@RafaelKayumov RafaelKayumov merged commit 6a0f960 into trunk Nov 6, 2025
19 of 20 checks passed
@RafaelKayumov RafaelKayumov deleted the WOOMOB-1616-navigate-to-booking-order branch November 6, 2025 13:43
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