-
Notifications
You must be signed in to change notification settings - Fork 574
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
Update swagger UI to 5.10.3 #26960
Update swagger UI to 5.10.3 #26960
Conversation
#build |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=c34fe8a2-7a74-47fc-b1b6-e66ac8e62283 Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_nqZRgIPFEe6tffXfLjKGSA Target locations of links might be accessible only to IBM employees. |
The build abutch3r-26960-20231115-0756 For help analyzing your personal build, go to https://libh-proxy1.fyre.ibm.com/cognitive/buildAnalysis.html?uuid=_nqZRgIPFEe6tffXfLjKGSA |
From the build results it looks like they have changed the way the lock icons are rendered such that the From manual testing, the OAS version has disappeared. This looks to be due to it having been moved to a separate component. Our |
2ce6976
to
687afdb
Compare
#libby |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=1ae1b1b8-ceaa-4b8a-9ddc-1708617ddbc8 Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_nEG8kIWREe6tffXfLjKGSA Target locations of links might be accessible only to IBM employees. |
a64e10e
to
17cb0f8
Compare
#libby |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=aa5fc575-89df-412c-97d4-b1e4f58267b3 Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_mQ518IkeEe6PKt0UtTmx2A Target locations of links might be accessible only to IBM employees. |
Open Liberty UI Manual TestingBrowser and version tested: Firefox 119.01 MicroProfile UI
Openapi 3.x UIPublic UI
Private UI
Functionality TestingMicroProfile UIAccess https://localhost:9443/openapi/ui
OpenAPI 3.x UIAccess https://localhost:9443/api/explorer
Customer provided customizationMicroProfile UI
OpenAPI-3.x UI
Endpoint customization(Microprofile only)
|
The build abutch3r-26960-20231122-0313 For help analyzing your personal build, go to https://libh-proxy1.fyre.ibm.com/cognitive/buildAnalysis.html?uuid=_mQ518IkeEe6PKt0UtTmx2A |
#libby |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=8f9c3579-29cc-42dd-98ee-8cfcb3e01a10 Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_8be8sIoZEe6PKt0UtTmx2A Target locations of links might be accessible only to IBM employees. |
The build abutch3r-26960-20231123-0916 For help analyzing your personal build, go to https://libh-proxy1.fyre.ibm.com/cognitive/buildAnalysis.html?uuid=_8be8sIoZEe6PKt0UtTmx2A |
Inspecting the behavior of the page, the issue is due to an initial load issue within Swagger-UI no longer adding the respective classes to the button element. If you cycle through a login and logout cycle you can see the |
e87097a
to
dc4c0c0
Compare
|
Fix checkout deprecation to support node16 Have the action commit to the current branch rather than opening a pull request if the branch is not integration. This makes it easier to create a branch with the required code changes and then use the action to rebuild them in the same PR. Update the action to only copy scss files from the swagger-ui source.
Update OpenAPI UI tests for swagger-ui 5.10.3 The page structure has changed slightly. Update the corresponding parts of the tests. SwaggerUI have changed how the OAS version is rendered and our replacement info component needs to be updated to match.
dc4c0c0
to
525e53b
Compare
#libby |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=27415687-87ef-4f04-bd95-4f661e73203d Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_I9-FII61Ee6gttl9kBC4bw Target locations of links might be accessible only to IBM employees. |
525e53b
to
59dbded
Compare
The build abutch3r-26960-20231129-0826 |
#libby |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=1838642859 Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_YzSgQZKgEe6WTPpngIn9MA Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=1838642859 has errors/failures. |
CI failures relate solely to springboot tests, all the openapi tests and in particular the full fat selenium tests passed |
The build abutch3r-26960-20231204-0535 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, though in future I'd prefer to see the manual changes in separate commits from the code being rebuilt at the end.
Particularly in this case, we've committed two versions of the built JS in the repository history.
Still needs the manual test runs on the output of the personal build.
#build |
Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_p1bekJRAEe6WTPpngIn9MA Target locations of links might be accessible only to IBM employees. |
Your personal build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=1843073955 Target locations of links might be accessible only to IBM employees. |
The second set of changes to the JS relate to the changes allowed by the removal of packages for IE11. so was a result of those changes. but we shouldn't see the need to commit two versions of the js again |
https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=1843073955 has errors/failures. |
The build abutch3r-26960-20231206-0719 For help analyzing your personal build, go to https://libh-proxy1.fyre.ibm.com/cognitive/buildAnalysis.html?uuid=_p1bekJRAEe6WTPpngIn9MA |
Manual testing of build image Browser and version tested on: Firefox 120.0 MicroProfile UI
Openapi 3.x UIPublic UI
Private UI
Functionality TestingMicroProfile UIAccess https://localhost:9443/openapi/ui
OpenAPI 3.x UIAccess https://localhost:9443/api/explorer
Customer provided customizationMicroProfile UI
OpenAPI-3.x UI
Endpoint customization(Microprofile only)
|
RTC and CI build failures do not relate at all to openapi and in particular the UI |
Update swagger UI 5.10.3 and update associated dependencies.
Update openapi-ui github action to work with Swagger-ui v5 and resolves node12 deprecations
Fixes #26948
Fixes #24487