Fix test app API #321
Fix test app API #321
Conversation
| sylius.security.api_regex: "^/api" | ||
| sylius.security.shop_regex: "^/(?!%sylius_admin.path_name%|new-api|api/.*|api$|media/.*)[^/]++" | ||
| sylius.security.new_api_route: "/new-api" | ||
| sylius.security.shop_regex: "^/(?!%sylius_admin.path_name%|api/.*|api$|media/.*)[^/]++" | ||
| sylius.security.new_api_route: "/api/v2" | ||
| sylius.security.new_api_regex: "^%sylius.security.new_api_route%" | ||
| sylius.security.new_api_admin_route: "%sylius.security.new_api_route%/admin" | ||
| sylius.security.new_api_admin_regex: "^%sylius.security.new_api_admin_route%" | ||
| sylius.security.new_api_shop_route: "%sylius.security.new_api_route%/shop" | ||
| sylius.security.new_api_shop_regex: "^%sylius.security.new_api_shop_route%" | ||
| sylius.security.new_api_user_account_route: "%sylius.security.new_api_shop_route%/account" | ||
| sylius.security.new_api_user_account_regex: "^%sylius.security.new_api_user_account_route%" |
There was a problem hiding this comment.
There should be no further need to define these parameters as there was extracted to bundles in Sylius in 1.9 version I guess
There was a problem hiding this comment.
I simply copied it from Sylius-Standard.
There was a problem hiding this comment.
I will check if they're necessary and remove if not.
There was a problem hiding this comment.
You are right @GSadee, all those parameters are already defined in bundles, so I removed them from the test app.
Co-authored-by: Grzegorz Sadowski <sadowskigp@gmail.com>
|
The base of this pull-request was changed, you need fetch and reset your local branch Unless you added new commits (to this branch) locally that you did not push yet, Feel free to ask for assistance when you get stuck 👍 |
|
Thanks, Victor! 🎉 |
Fix and enable the API in test app.