Skip to content

[WOOMOB-2454] Improve POS catalog syncing loading UX#15505

Merged
samiuelson merged 2 commits intotrunkfrom
woomob-2454-woo-pos-syncing-can-take-5-minutes-on-large-store
Mar 10, 2026
Merged

[WOOMOB-2454] Improve POS catalog syncing loading UX#15505
samiuelson merged 2 commits intotrunkfrom
woomob-2454-woo-pos-syncing-can-take-5-minutes-on-large-store

Conversation

@malinajirka
Copy link
Copy Markdown
Contributor

@malinajirka malinajirka commented Mar 10, 2026

Description

Fixes WOOMOB-2454

Improves the POS catalog syncing screen UX for large stores where syncing can take several minutes. Adds a hint message below the exit button explaining that syncing may take a few minutes and that users can leave.

Also enables the file-based local catalog sync feature flag on debug builds.

Test Steps

  1. Open POS on a debug build
  2. Observe the syncing screen shows "Syncing catalog" with a spinner (if the POS opens immediately, you might need to delete your data or switch to a store you haven't synced locally yet)
  3. Verify the hint text below the Exit POS button reads "Catalog syncing may take a few minutes." and "You can leave and syncing will continue in the background."
  4. Tap "Exit POS" and confirm it exits correctly

Images/gif

Screenshot 2026-03-10 at 15 18 52
  • 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.

Refactor layout to include a new column for syncing hints and subtitles, with updated text styles and padding for better clarity and alignment.
@malinajirka malinajirka added feature: point of sale POS project status: feature-flagged Behind a feature flag. Milestone is not strongly held. type: enhancement A request for an enhancement. labels Mar 10, 2026
@malinajirka malinajirka added this to the 24.4 milestone Mar 10, 2026
@malinajirka malinajirka marked this pull request as ready for review March 10, 2026 14:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves the Woo POS catalog syncing splash UX for long-running syncs by adding clearer messaging that syncing may take a few minutes and can continue in the background, and enables the local catalog file-based approach feature flag by default on debug builds.

Changes:

  • Added a new hint string and updated the syncing subtitle messaging.
  • Updated the POS syncing splash UI to display the new hint + subtitle beneath the “Exit POS” button.
  • Enabled WOO_POS_LOCAL_CATALOG_FILE_APPROACH by default on debug builds.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
WooCommerce/src/main/res/values/strings.xml Adds a new syncing hint string and updates the syncing subtitle copy.
WooCommerce/src/main/kotlin/com/woocommerce/android/util/FeatureFlag.kt Enables the local catalog file-based approach flag on debug builds by default.
WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/splash/WooPosSplashScreen.kt Adjusts the syncing splash UI to show the new hint text under the exit action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wpmobilebot
Copy link
Copy Markdown
Collaborator

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

App NameWooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Build Number734
Version24.3-rc-3
Application IDcom.woocommerce.android.prealpha
Commitcda1dad
Installation URL3n4vfjtdn3058
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.55%. Comparing base (ecb8a65) to head (cda1dad).
⚠️ Report is 137 commits behind head on trunk.

Additional details and impacted files
@@            Coverage Diff            @@
##              trunk   #15505   +/-   ##
=========================================
  Coverage     39.55%   39.55%           
+ Complexity    11235    11234    -1     
=========================================
  Files          2252     2252           
  Lines        129662   129662           
  Branches      18165    18165           
=========================================
  Hits          51290    51290           
+ Misses        73134    73133    -1     
- Partials       5238     5239    +1     

☔ 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.

Copy link
Copy Markdown
Contributor

@samiuelson samiuelson left a comment

Choose a reason for hiding this comment

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

LGTM!

@samiuelson samiuelson merged commit c65787e into trunk Mar 10, 2026
29 checks passed
@samiuelson samiuelson deleted the woomob-2454-woo-pos-syncing-can-take-5-minutes-on-large-store branch March 10, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: point of sale POS project status: feature-flagged Behind a feature flag. Milestone is not strongly held. type: enhancement A request for an enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants