Skip to content

Conversation

@miborra
Copy link
Contributor

@miborra miborra commented Jul 13, 2016

What's this PR do/fix?

This PR modifies how to set jsonEditor parameter to solve problem related on #1373 task

Are there unit tests? If not how should this be manually tested?

No there aren't. To test it you should configure UiConfiguration with enableJsonEditor parameter to false. In swagger page, the button Try it! shouldn't be accesible.

Any background context you want to provide?

No

What are the relevant issues?

#1373

@dilipkrish dilipkrish merged commit fb305ad into springfox:master Jul 13, 2016
@dilipkrish dilipkrish added the PR label Jul 13, 2016
@dilipkrish dilipkrish added this to the 2.6.0 milestone Jul 13, 2016
@dilipkrish
Copy link
Member

👍 thanks for catching that! Do you know if this is a problem in all browsers or just a particular browser?

@miborra
Copy link
Contributor Author

miborra commented Jul 14, 2016

In all browsers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants