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

Added regex to replace / with _ in resource name #1

Merged
merged 1 commit into from
Sep 28, 2011

Conversation

jdpgrailsdev
Copy link

I found an issue with the Swagger UI with resources with forward slashes in their path. For instance, if you have a resource bound to a nested URI path, such as /widgets/widget, the Swagger UI cannot handle this, as forward slashes are not allowed in DOM ID values (it causes the click handlers to expand the widgets on the UI to not work). Therefore, I applied a simple fix to replace all instances of '/' with '_' to get the UI working again.

fehguy added a commit that referenced this pull request Sep 28, 2011
Added regex to replace / with _ in resource name thanks @jdpgrailsdev!
@fehguy fehguy merged commit 4f8b2cd into swagger-api:master Sep 28, 2011
fehguy pushed a commit that referenced this pull request Nov 4, 2013
Update to latest swagger-ui code
cguillot pushed a commit to cguillot/swagger-ui that referenced this pull request Nov 19, 2014
shelljs (https://www.npmjs.org/package/shelljs) allows unix command to be executed on Windows.

swagger-apigh-1
cguillot pushed a commit to cguillot/swagger-ui that referenced this pull request Nov 19, 2014
Replace execution of unix related operations such as cp, rm, and cat with shelljs' methods to support cross platform.

swagger-apigh-1
cguillot pushed a commit to cguillot/swagger-ui that referenced this pull request Nov 19, 2014
Without -f content of dist will remain the same.

swagger-apigh-1
cguillot pushed a commit to cguillot/swagger-ui that referenced this pull request Nov 19, 2014
cguillot pushed a commit to cguillot/swagger-ui that referenced this pull request Nov 19, 2014
gwicke added a commit to gwicke/swagger-ui that referenced this pull request Mar 9, 2015
Support RFC 6570 optional path segments
russell pushed a commit to russell/swagger-ui that referenced this pull request Jun 28, 2015
Changed try button to be shown by default
fehguy pushed a commit that referenced this pull request Dec 16, 2015
Fix $ref strings treated as objects by JSONEditor
fehguy pushed a commit that referenced this pull request Feb 16, 2016
mkrufky pushed a commit to mkrufky/swagger-ui that referenced this pull request Feb 26, 2016
zyoung51 pushed a commit to zyoung51/swagger-ui that referenced this pull request Mar 2, 2016
vincent-zurczak referenced this pull request in roboconf/swagger-ui Aug 18, 2016
Update to latest swagger-ui code
vincent-zurczak referenced this pull request in roboconf/swagger-ui Aug 19, 2016
Fix $ref strings treated as objects by JSONEditor
vincent-zurczak referenced this pull request in roboconf/swagger-ui Aug 19, 2016
banders referenced this pull request in banders/swagger-ui Feb 3, 2017
lshaw-sb referenced this pull request in SmartBear/swagger-ui May 10, 2019
…ult-margins-padding

Reset default text margins, Add updated spacing to information-container, scheme-container
eLvErDe pushed a commit to eLvErDe/swagger-ui that referenced this pull request Dec 3, 2019
char0n added a commit that referenced this pull request Jul 14, 2021
char0n added a commit that referenced this pull request Jul 30, 2021
char0n added a commit that referenced this pull request Jul 30, 2021
char0n added a commit that referenced this pull request Aug 11, 2021
char0n added a commit that referenced this pull request Sep 10, 2021
char0n added a commit that referenced this pull request Sep 13, 2021
char0n added a commit that referenced this pull request Sep 13, 2021
char0n added a commit that referenced this pull request Sep 15, 2021
char0n added a commit that referenced this pull request Sep 20, 2021
char0n added a commit that referenced this pull request Oct 8, 2021
char0n added a commit that referenced this pull request Oct 12, 2021
char0n added a commit that referenced this pull request Oct 22, 2021
char0n added a commit that referenced this pull request Nov 2, 2021
char0n added a commit that referenced this pull request Nov 3, 2021
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

Successfully merging this pull request may close these issues.

2 participants