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

Dynamic items sold view/column/button #14121

Closed
mikejolley opened this Issue Apr 7, 2017 · 10 comments

Comments

Projects
None yet
4 participants
@mikejolley
Member

mikejolley commented Apr 7, 2017

The items side column was removed in 3.0 due to inefficiencies and heavy load (it loaded all items for all orders).

http://ideas.woocommerce.com/forums/133476-woocommerce/suggestions/18832222-dynamically-display-order-content-on-orders-page

Build a dynamic version which uses AJAX to pull a list of items and display them somewhere, somehow.

(Snippet for adding it back through code in the meantime, but not recommended due to the performance issues: https://gist.github.com/rynaldos/4d193083e0ca7bb187d361deae837f04)

@pmgarman

This comment has been minimized.

Show comment
Hide comment
@pmgarman

pmgarman Apr 7, 2017

Contributor

Have any thoughts on if this loads on page load for all orders (giving the ability to see it without interaction) or on hover on the order? Based on user feedback from having removed it any insights into how people were using it most that could guide that?

Loading immediately keeps it easy to view on the fly without interaction but adds additional load for all page loads, but adds additional (potentially unneeded) load by triggering Ajax calls on every page load of the orders screen.

Contributor

pmgarman commented Apr 7, 2017

Have any thoughts on if this loads on page load for all orders (giving the ability to see it without interaction) or on hover on the order? Based on user feedback from having removed it any insights into how people were using it most that could guide that?

Loading immediately keeps it easy to view on the fly without interaction but adds additional load for all page loads, but adds additional (potentially unneeded) load by triggering Ajax calls on every page load of the orders screen.

@mikejolley

This comment has been minimized.

Show comment
Hide comment
@mikejolley

mikejolley Apr 8, 2017

Member

@pmgarman Previous version was displayed on click. It would have to be ajaxified to avoid the large number of item queries we removed the original column for.

Member

mikejolley commented Apr 8, 2017

@pmgarman Previous version was displayed on click. It would have to be ajaxified to avoid the large number of item queries we removed the original column for.

@mikejolley

This comment has been minimized.

Show comment
Hide comment
@mikejolley

mikejolley Apr 20, 2017

Member

@jameskoster would you have time to whip up a wireframe for this? Whilst the issue to is show line items, it could be some extra details too - think of it as a quick view of the main order details you can access via a button on the order list.

Member

mikejolley commented Apr 20, 2017

@jameskoster would you have time to whip up a wireframe for this? Whilst the issue to is show line items, it could be some extra details too - think of it as a quick view of the main order details you can access via a button on the order list.

@jameskoster

This comment has been minimized.

Show comment
Hide comment
@jameskoster

jameskoster Apr 21, 2017

Member

We're looking at orders next in c9n. Quick view might not be as relevant there as the app is faster but if it is then the same design could be applied to core. If not, I can look at this separately.

Should be ready in a couple of weeks. Hopefully we can make it work with your Sprint 4 deadline.

Member

jameskoster commented Apr 21, 2017

We're looking at orders next in c9n. Quick view might not be as relevant there as the app is faster but if it is then the same design could be applied to core. If not, I can look at this separately.

Should be ready in a couple of weeks. Hopefully we can make it work with your Sprint 4 deadline.

@mikejolley

This comment has been minimized.

Show comment
Hide comment
@mikejolley

mikejolley May 11, 2017

Member

@jameskoster any movement there? If not we can push it back to 3.2 since we have quite a lot left to do anyway.

Member

mikejolley commented May 11, 2017

@jameskoster any movement there? If not we can push it back to 3.2 since we have quite a lot left to do anyway.

@jameskoster

This comment has been minimized.

Show comment
Hide comment
@jameskoster

jameskoster May 11, 2017

Member

We're finishing up the designs / prototypes this week. As yet, being able to view order items on the order list screen hasn't come up in any of our stories/flows. Ultimately this feels more like a performance issue than a UI issue.

Member

jameskoster commented May 11, 2017

We're finishing up the designs / prototypes this week. As yet, being able to view order items on the order list screen hasn't come up in any of our stories/flows. Ultimately this feels more like a performance issue than a UI issue.

@mikejolley mikejolley modified the milestone: 3.2.0 Jun 29, 2017

@mikejolley mikejolley modified the milestone: 3.2.0 Jul 6, 2017

@mikejolley

This comment has been minimized.

Show comment
Hide comment
@pipdig

This comment has been minimized.

Show comment
Hide comment
@pipdig

pipdig Jul 14, 2017

If anyone wants a quick solution in the meantime, we created a quick plugin for our own site which displays the ordered items via an ajax request - https://wordpress.org/plugins/purchased-items-column-woocommerce

pipdig commented Jul 14, 2017

If anyone wants a quick solution in the meantime, we created a quick plugin for our own site which displays the ordered items via an ajax request - https://wordpress.org/plugins/purchased-items-column-woocommerce

@mikejolley

This comment has been minimized.

Show comment
Hide comment
@mikejolley

mikejolley Sep 4, 2017

Member

This is being worked on in branch: #16700

Member

mikejolley commented Sep 4, 2017

This is being worked on in branch: #16700

@mikejolley mikejolley closed this Sep 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment