Skip to content

Variations: Show loading indicators #8573

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jan 10, 2023
Merged

Variations: Show loading indicators #8573

merged 2 commits into from
Jan 10, 2023
+55 −9

Conversation

@Ecarrion
Copy link
Contributor

@Ecarrion Ecarrion commented Jan 6, 2023

Closes: #8491

Why

This PRs shows a blocking loading indicator while variations are being fetched and created so the merchant is informed about what is going on.

How

  • Update InProgressViewController to allow for a runtime update.
  • Update ProductVariationsViewController to show and hide the "Fetching..." and "Creating..." blocking indicators.

Demo

demo.mov

Testing Steps

  • Navigate to a variable product
  • Make sure there are some variations to be created
  • Start the "Generate All Variations" process
  • See that proper loading indicators are being shown.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@Ecarrion Ecarrion added the feature: variation list Related to the variations list for variable products. label Jan 6, 2023
@Ecarrion Ecarrion added this to the 11.8 milestone Jan 6, 2023
@Ecarrion Ecarrion requested a review from ealeksandrov January 6, 2023 16:10
@peril-woocommerce
Copy link

peril-woocommerce bot commented Jan 6, 2023

Warnings
⚠️ This PR is assigned to a milestone which is closing in less than 2 days Please, make sure to get it merged by then or assign it to a later expiring milestone

Generated by 🚫 dangerJS

@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jan 6, 2023

You can test the changes from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr8573-3616738 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@spencertransier spencertransier modified the milestones: 11.8, 11.9 Jan 7, 2023
ealeksandrov
ealeksandrov approved these changes Jan 9, 2023
View reviewed changes
Copy link
Contributor

@ealeksandrov ealeksandrov left a comment

Choose a reason for hiding this comment

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

:shipit:

@Ecarrion Ecarrion merged commit ab8f07b into trunk Jan 10, 2023
@Ecarrion Ecarrion deleted the issue/8491-loading-indicators branch January 10, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ealeksandrov ealeksandrov ealeksandrov approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

feature: variation list Related to the variations list for variable products.

Projects

None yet

Milestone

11.9

Development

Successfully merging this pull request may close these issues.

Variations: Inform users about fetching & creating loading states

5 participants

@Ecarrion @wpmobilebot @ealeksandrov @spencertransier