Skip to content

Comments

[WOOMOB-2286] Change POS bookings sort to oldest-to-newest#15403

Open
malinajirka wants to merge 3 commits intotrunkfrom
issue/woomob-2286-pos-bookings-no-order-selected-upon-opening-pos-bookings
Open

[WOOMOB-2286] Change POS bookings sort to oldest-to-newest#15403
malinajirka wants to merge 3 commits intotrunkfrom
issue/woomob-2286-pos-bookings-no-order-selected-upon-opening-pos-bookings

Conversation

@malinajirka
Copy link
Contributor

@malinajirka malinajirka commented Feb 20, 2026

Fixes WOOMOB-2286

Description

Changed POS bookings list sorting from newest-to-oldest to oldest-to-newest. The POS and main Bookings tab share the same Room Bookings table. When they used different sort orders, cached data from one screen would briefly display in the wrong order when opening the other, causing a visible "jump" as the fresh fetch replaced it. Switching POS to oldest-to-newest aligns with the main Bookings tab and matches user expectations (start of day → end of day).

Test Steps

  1. Open POS → Bookings — verify bookings are sorted from earliest to latest start time
  2. Restart the app → open POS → Bookings — verify no visual content jump when fresh data loads
  3. Compare the order with the main app Bookings tab — they should match
  • I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary.

@malinajirka malinajirka added this to the 24.2 milestone Feb 20, 2026
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 20, 2026

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 Number727
Version24.1-rc-3
Application IDcom.woocommerce.android.prealpha
Commit93788e9
Installation URL6qdslb289jtd0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.29%. Comparing base (ab7c42b) to head (93788e9).
⚠️ Report is 26 commits behind head on trunk.

Additional details and impacted files
@@            Coverage Diff            @@
##              trunk   #15403   +/-   ##
=========================================
  Coverage     39.29%   39.29%           
  Complexity    10977    10977           
=========================================
  Files          2238     2238           
  Lines        127540   127540           
  Branches      17802    17802           
=========================================
  Hits          50113    50113           
  Misses        72318    72318           
  Partials       5109     5109           

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

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