-
-
Notifications
You must be signed in to change notification settings - Fork 321
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
groups.order
now supports *
wildcard, so you can put groups at the end
#731
groups.order
now supports *
wildcard, so you can put groups at the end
#731
Conversation
Makes sense (and no new config items😆); I like.👍 |
Question: did you find it difficult to contribute (is knowing where to look, what goes where, how to add tests, etc)? |
Just been playing around with the current implementation, it loses the natural order sort so I'll have a re think on this |
There's always been tests related to what I've changed to it's easy to find them. In terms of the code, my ide gets me around fast so I don't really have to spend time looking for things that much |
…to determine where in the ordering all unspecified groups will be placed
9223248
to
56d6735
Compare
This new implementation retains the natural sorting functionality for the wildcard groups |
Btw, does it also work for subgroups?
|
Could you send a PR to the docs repo as well? https://github.com/knuckleswtf/scribe-docs I think the places you need to update are https://scribe.knuckles.wtf/laravel/reference/config#order and https://scribe.knuckles.wtf/laravel/tasks/sorting-and-inheritance#sorting-endpoints-and-groups. Can probably just edit them from the GitHub UI. |
Not currently, it seems the sub group sorting is done differently to the top level groups. We could look at adding support for that later? |
I've created knuckleswtf/scribe-docs#25 for the documentation. I'm having trouble getting docusaurus to start however. So I have not been able to confirm what this looks like. Below is the error I get running
|
groups.order
now supports *
wildcard, so you can put groups at the end
No worries, that's JavaScript for you. Top-level support is fine for now. |
…to determine where in the ordering all unspecified groups will be placed.
Opening this one as a Draft so we can talk about implementation/config @shalvah. Hopefully this is a feaure Scribe can support.
This PR is adding a feature which allows you to bury endpoints at the bottom of the list for example older versions that you don't want consumers going to first.
This is done with the following config:
This will generate the groups like the screenshot below: