Skip to content
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

Order Status count badge #1217

Closed
jonleverrier opened this issue Jan 16, 2020 · 3 comments
Closed

Order Status count badge #1217

jonleverrier opened this issue Jan 16, 2020 · 3 comments
Labels
💡 enhancement Ideas and suggestions

Comments

@jonleverrier
Copy link

jonleverrier commented Jan 16, 2020

Description

A small point! In the CP, when moving orders around via order status, the count badge only reports the correct number of orders in it once the page has been refreshed manually, even though the orders get moved between statuses instantly.

Steps to reproduce

  1. In orders, click into an order status with orders in it
  2. Tick the checkbox of an order, click on the cog icon at the top and select "Update Order Status"
  3. Move the order into another order status
  4. Orders are moved instantly, but the count badge reports an incorrect number until you refresh the page manually

Additional info

  • Craft version: 3.4.0-RC1.1
  • PHP version: 7.3
  • Database driver & version: MySQL 5.5.5
  • Plugins & versions: Craft Commerce 3.0.0-RC1
@jonleverrier jonleverrier changed the title 3.0-RC1 Order Status count badge Order Status count badge Jan 16, 2020
@lukeholder lukeholder added the 💡 enhancement Ideas and suggestions label Jan 17, 2020
@jonleverrier
Copy link
Author

jonleverrier commented Feb 21, 2020

@lukeholder Do you think this enhancement will be worked on in the next release?

As small as it seems, I have a client i'm moving across to Commerce and I know it's something they're going to mention as they rely heavily on order statuses.

Certainly not a deal breaker, but would be good to set their expectation.

@nfourtythree
Copy link
Contributor

Hi,

This has been updated in 3df9a68

It will be included in the next release.

Thanks.

@jonleverrier
Copy link
Author

@nfourtythree Thank you for looking at this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 enhancement Ideas and suggestions
Projects
None yet
Development

No branches or pull requests

3 participants