solidus icon indicating copy to clipboard operation
solidus copied to clipboard

Scope Admin Orders to Store

Open jarednorman opened this issue 2 years ago • 2 comments

Desired Behavior

When viewing the admin orders index, I should see only the orders for the current store.

Current Behavior

When viewing the admin orders index, I see all orders for all stores.

(Optional) Additional notes

I'm curious where we want to draw the line with multi-store support. What belongs in core and what belongs in solidus_multi_domain (which isn't in great shape at the moment.)

Given that the new admin shows the current store and orders are one of only a couple models that are tied to the store, should we be scoping the visible orders down to that store? It seems like it would be trivial to change by altering this line.

Another option would be adding the current store to the default search query on that page.

Thoughts?

jarednorman avatar Jan 23 '24 20:01 jarednorman

Hey @jarednorman, thanks for reporting. How is it working currently on the legacy admin? Is the feature you are describing always available, or only combined with solidus_multi_domain?

kennyadsl avatar Jan 24 '24 09:01 kennyadsl

I've not got a store handy to test whether it works on solidus_multi_domain, but the legacy admin doesn't do it. This would be a functional change in the admin. I can see arguments both ways, but mainly I thought it seemed weird that the new admin navigation shows you what store you are viewing... but then doesn't scope the orders (or literally anything, as far as I know) down to that store.

jarednorman avatar Jan 25 '24 16:01 jarednorman