Add a page to display all Recurrence rules

[vladimir-8]

includes:

• Recurrence rules ("recurring orders") created via the Delivery form
• Recurrence rules created via the Dispatch dashboard (possible options: Order+Delivery; Delivery without an Order/price; one/many Tasks without a Delivery)

[Atala]

I made it crash :

• go to the dispatch and create a delivery rule with just 1 dropoff
• go to recurrence rules list page for the store

[Atala]

it does seem to me that the dropdown list to select a store when creating a recurrent rule in the dispatch does not display all the stores. is that expected? i can look deeper

[vladimir-8]

in french breadcrumb it is still abonnements ("subscription")

Are you sure it was the right branch? Looks good on my machine:

[vladimir-8]

i am not 100% convinced to redirect the user to the "deliveries pages" when you have succesfully edited a subscription (you dont have feedback on what happenned), maybe redirect to the subscription list page or just add a flash message? (so the user knows the change has been taken into account)

In a normal flow, it goes to the previous page (the list of recurrence rules, if you opened a recurrence rule from there), see RedirectToRefererType. It goes to the "deliveries pages" only as a fallback it it does not know what the previous page is. It happens if you refresh a page, for example.

[vladimir-8]

actually i thought the recurrence rules list page would be in the sidebar on "http://localhost/admin/deliveries" not on the store level. i guess admin users would like to see all the subscriptions for all his store. what do you think?

Yeah, I discussed it with Paul and we decided to put it on a store level for now. Maybe, at some point, we will make a global one as well

[vladimir-8]

it does seem to me that the dropdown list to select a store when creating a recurrent rule in the dispatch does not display all the stores. is that expected? i can look deeper

For me, it displays everything. I'd say don't spend time on it, as we want to migrate away from that flow

[vladimir-8]

I made it crash :

go to the dispatch and create a delivery rule with just 1 dropoff
go to recurrence rules list page for the store

It seems you were not running the latest version locally. The code in your stack trace is different from the last commit in this branch.

Now it handles both one pickup or one dropoff: