- 🇳🇱Netherlands huijse
Patch #4 works.
Test result:
With: administer commerce_subscription
You cannot access the subscription type configuration, but can see the subscription overview on:
admin/commerce/subscriptions which was the point.With permission: administer commerce_subscription_types
You get access to: admin/commerce/config/subscriptions/subscription-typesWhich is needed for admins to make configurations but not subscription managers.
- Issue was unassigned.
- 🇺🇸United States DamienMcKenna NH, USA
As a reminder - when you finish your work on an issue please set the "Assigned" field back to "Unassigned".
-
jsacksick →
committed 286c0717 on 8.x-1.x
Issue #3211471 by recidive, mhawwari, jsacksick: Add "Administer...
-
jsacksick →
committed 286c0717 on 8.x-1.x
- Status changed to Fixed
5 months ago 10:59am 28 June 2024 - 🇮🇱Israel jsacksick
For consistency sake, renamed the permission to "administer commerce_subscription_type". (this is what we do for order types and product types).
However, for product types and order types, the path is: admin/commerce/config/order-types, and admin/commerce/config/product-types. If we remove "subscriptions" from the path then the breadcrumb doesn't append the parent "Subscriptions" config page...So keeping the path as in the patch, but also wondering if we shouldn't change it to:
/admin/commerce/config/subscriptions/types. I don't think there's a need to repeat subscription twice in the path... But could be a followup. Automatically closed - issue fixed for 2 weeks with no activity.