Closed (fixed)
Project:
Commerce Core
Version:
8.x-2.x-dev
Component:
Order
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Jan 2016 at 18:46 UTC
Updated:
4 Feb 2016 at 15:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
bojanz commentedComment #3
goz commentedComment #4
goz commentedThis patch uses state views filter added by https://www.drupal.org/node/2648684
Remaining tasks :
Comment #5
mglamanMarking needs work while we wait on #2648684: Use InOperator filter for state views filter
Comment #6
goz commented#2648684 is fixed and commited.
Comment #7
mglamanWe should drop the "created" column as the only timestamp relevant to the customer should be "placed"
Comment #8
bojanz commentedWe also want to show the store, but hide it if there's only 1, just like the order admin view does.
Comment #9
mglamanMade a PR: https://github.com/drupalcommerce/commerce/pull/305
Takes #4 but removes created field. Adds a test.
Comment #10
aaronchristian commentedI've attached 2 different variations. One as a default and another concept for orders with shippable goods.
Let me know what you guys think!
Comment #11
bojanz commentedWaiting on a rebase.
Comment #12
goz commentedSkip the Store field asked by Bojan in #8 until we actually have a Stores page.
Comment #13
goz commentedComment #14
mglamanRebased
Comment #16
bojanz commentedI committed just the view, with the cart filter removed (cause it was unneeded and orders can't assume the existence of carts).
The test can't work because it tests access controls that the view doesn't have yet, Tavi is working on that.
Here's the removed code: