-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: how to add new admin route under existing route #10652
docs: how to add new admin route under existing route #10652
Conversation
|
@eugenepro2 is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for opening this PR! Just need a few changes
@@ -145,6 +145,32 @@ Some caveats for nested UI routes in the sidebar: | |||
- Nested routes in setting pages aren't shown in the sidebar to follow the admin's design conventions. | |||
- The `icon` configuration is ignored for the sidebar item of nested UI route to follow the admin's design conventions. | |||
|
|||
|
|||
### Route Under Existing Admin Route | |||
You can locate a route under an existing route. For example, you can locate a route under the orders route in the sidebar. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can locate a route under an existing route. For example, you can locate a route under the orders route in the sidebar. | |
You can add a custom UI route under an existing route. For example, you can add a route under the orders route: |
@shahednasser Thanks for your suggestions. I've committed this changes. Please have a look. |
Co-authored-by: Shahed Nasser <shahednasser@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks! Will merge this once the next release is out.
@shahednasser can you merge it? |
* docs: how to add new admin route under existing route * docs: update about defineRouteConfig * docs: add new line Co-authored-by: Shahed Nasser <shahednasser@gmail.com> * docs: add new line --------- Co-authored-by: Shahed Nasser <shahednasser@gmail.com>
* docs: how to add new admin route under existing route * docs: update about defineRouteConfig * docs: add new line Co-authored-by: Shahed Nasser <shahednasser@gmail.com> * docs: add new line --------- Co-authored-by: Shahed Nasser <shahednasser@gmail.com>
This PR updated docs and describe how to add new admin route under existing route.
Ref: #10587 (comment)