Skip to content
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

Adding the endpoint to the navigation #199

Closed
Szasza opened this issue Oct 11, 2017 · 3 comments
Closed

Adding the endpoint to the navigation #199

Szasza opened this issue Oct 11, 2017 · 3 comments

Comments

@Szasza
Copy link
Contributor

Szasza commented Oct 11, 2017

Current situation: in the left-hand side navigation all the menu items are listed based on endpoints and request types, but it is the summary which is shown next to the request type.

Example:
GET A test summary here

Question: is it an acceptable change to add the endpoint path to the left-hand side nav's menu items and to move the summary into a next line?

Example:

GET /some/endpoint
A test summary here

If so, I am happy to create a PR.

@Szasza
Copy link
Contributor Author

Szasza commented Oct 12, 2017

Note to self: code changes are implemented but waiting for #200 to be checked and merged first to avoid issues.

@brendo
Copy link
Contributor

brendo commented Oct 18, 2017

How do you feel about having this behind an option? Having a large API could make the navigation a little unwieldy to navigate, but I agree that sometimes it's easier to just look POST /pets than Add a Pet

@Szasza
Copy link
Contributor Author

Szasza commented Oct 18, 2017

Sure, can do, actually the branch I have for this shows both for one nav item, i.e.

POST /pets
Add a pet

Happy to make it configurable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants