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

Sort routes alphabetically #254

Closed

Conversation

bpruitt-goddard
Copy link

As the title implies, sort the routes alphabetically. Currently, they are displayed in whatever order they are sent in.

@fehguy
Copy link
Contributor

fehguy commented Jul 7, 2013

Hi thanks for sending, but I think it's best if they're displayed in the order sent, so the sender has control over the display order.

@fehguy fehguy closed this Jul 7, 2013
@filip26
Copy link

filip26 commented Nov 7, 2013

What about make it optional? I can imagine option sortAlphabetically=true|false. Note that keep alphabetical order on server is really painful now ... see swagger-api/swagger-core#115

@tagliola
Copy link

tagliola commented Jul 4, 2014

+1, alphabetically would eliminate a lot of tedious administration in our resources.

@fehguy
Copy link
Contributor

fehguy commented Jul 4, 2014

Hi @tagliola I'd like to make this a configuration for the next version of swagger-ui. I'll tag this issue appropriately.

@fehguy fehguy added the Feature label Jul 4, 2014
@fehguy fehguy reopened this Jul 4, 2014
@rage-shadowman
Copy link

You could make it configurable via a sorter param that takes a function much like java comparator. It would take 2 arguments and return a number less than, greater than or equal to zero depending on whether the first argument should come before, after or is comparably equal to the second argument.

chatch pushed a commit to chatch/swagger-ui that referenced this pull request Jul 20, 2014
@fehguy
Copy link
Contributor

fehguy commented Jul 28, 2014

see #496

@fehguy fehguy closed this Jul 28, 2014
shelbys added a commit to shelbys/swagger-ui that referenced this pull request Jul 30, 2014
* upstream/master:
  provide option sorter=[alpha|method]
  fixed undefined variable errors
  remove console.info
  swagger-api#254 alphabetical sort of apis and operations under apis new option sortAlphabetical=true|false
  updated distro to include swagger-api#493
  Fix for issue swagger-api#492; HTML in response headers
  switched to snippet view swagger-api#491
  updated versions, swagger-js

Conflicts:
	dist/swagger-ui.js
	dist/swagger-ui.min.js
	src/main/coffeescript/view/SignatureView.coffee
Alexgalinier pushed a commit to INTENT-TECHNOLOGIES/swagger-ui that referenced this pull request Nov 17, 2014
vincent-zurczak pushed a commit to roboconf/swagger-ui that referenced this pull request Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants