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

Expose the 'showCommonExtensions' configuration available in swagger-ui #2973

Merged
merged 1 commit into from Aug 3, 2019

Conversation

benbenw
Copy link
Contributor

@benbenw benbenw commented Apr 16, 2019

Co-authored-by: Ippei Nawate saku.ni63@gmail.com
Co-authored-by: Benoit b.wiart@ubik-ingenierie.com

What's this PR do/fix?

Expose the 'showCommonExtensions' configuration available in swagger-ui
through springfox UiConfiguration and UiConfigurationBuilder
It's available in swagger-ui since v3.14.0

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

Yes

Any background context you want to provide?

Based on PR #2663 from Ippei Nawate
Rebased for master
Add missing modification of springfox ui js and keep the backward compatibility by adding a new constructor in UiConfiguration

What are the relevant issues?

#2402

through springfox UiConfiguration and UiConfigurationBuilder

It's available in swagger-ui since v3.14.0

Co-authored-by: Ippei Nawate <saku.ni63@gmail.com>
Co-authored-by: Benoit <b.wiart@ubik-ingenierie.com>
@codecov
Copy link

codecov bot commented Apr 16, 2019

Codecov Report

Merging #2973 into master will decrease coverage by 0.05%.
The diff coverage is 54.54%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2973      +/-   ##
============================================
- Coverage     94.32%   94.26%   -0.06%     
- Complexity     3239     3241       +2     
============================================
  Files           364      364              
  Lines          8381     8392      +11     
  Branches        619      619              
============================================
+ Hits           7905     7911       +6     
- Misses          318      323       +5     
  Partials        158      158
Impacted Files Coverage Δ Complexity Δ
...umentation/swagger/web/UiConfigurationBuilder.java 100% <100%> (ø) 17 <1> (+1) ⬆️
...fox/documentation/swagger/web/UiConfiguration.java 85.33% <37.5%> (-5.72%) 23 <1> (+1)

@dilipkrish dilipkrish added the PR label Aug 3, 2019
@dilipkrish dilipkrish added this to the 3.0 milestone Aug 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants