Add support for nested groups to admin routing
What does this MR do?
Changes admin routing to work with wildcard namespace id. Since rails resources does not work well with /
inside *id
I need to replace resource with scope + match
combo. As result some routing naming changed because resource put action names as prefix to the path while match always does suffix.
As bonus I replaced admin_namespaces_projects_path
with admin_projects_path
.
Are there points in the code the reviewer needs to double check?
no
Why was this MR needed?
To make nested groups work in admin area
Screenshots (if relevant)
no
Does this MR meet the acceptance criteria?
Changelog entry addedDocumentation created/updatedAPI support added- Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
Merge request reports
Activity
mentioned in issue #2772 (closed)
added 1 commit
- 093f4050 - Add support for nested groups to admin routing
- Resolved by username-removed-444
- Resolved by username-removed-444
added 1 commit
- 6e36a9db - Add support for nested groups to admin routing
marked the task Squashed related commits together as completed
marked the task Conform by the style guides as completed
marked the task Conform by the merge request performance guides as completed
@DouweM please review
assigned to @DouweM
mentioned in merge request !7976 (closed)
added 34 commits
-
6e36a9db...3f768f9a - 33 commits from branch
master
- d8b7df3c - Add support for nested groups to admin routing
-
6e36a9db...3f768f9a - 33 commits from branch
assigned to @smcgivern
mentioned in commit b70b9628