Skip to content

[Watch] Fetch and display Order List Data #12807

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

Merged
merged 12 commits into from
May 22, 2024

Conversation

Ecarrion
Copy link
Contributor

@Ecarrion Ecarrion commented May 22, 2024

Closes: #12500

Why

This PR fetches and displays the first 50 orders in the watch app.

How

  • Reuse the OrdersRemote to fetch the data
  • Reuse OrderListCellViewModel to transform the data presentation
  • Adjust the necessary files to make them compile on whatchOS.

Demo

demo.mov

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@Ecarrion Ecarrion self-assigned this May 22, 2024
@Ecarrion Ecarrion added this to the 18.8 milestone May 22, 2024
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

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

App NameWooCommerce iOS WooCommerce iOS
Build Numberpr12807-b0d6a96
Version18.7
Bundle IDcom.automattic.alpha.woocommerce
Commitb0d6a96
App Center BuildWooCommerce - Prototype Builds #9183
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@Ecarrion Ecarrion merged commit 1f00db0 into ec/feature-watch-app May 22, 2024
27 checks passed
@Ecarrion Ecarrion deleted the issue/12500-order-list-fetch branch May 22, 2024 13:43
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.

3 participants