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

refactor(ui): update swagger-ui to v4 #2066

Merged
merged 3 commits into from
Jul 8, 2022
Merged

Conversation

volkflo
Copy link
Member

@volkflo volkflo commented May 23, 2022

No description provided.

@volkflo volkflo requested a review from msavy May 23, 2022 12:42
@volkflo
Copy link
Member Author

volkflo commented May 23, 2022

I think I got all problems fixed, have to triple check that.
But looks good atm.

@cypress
Copy link

cypress bot commented May 23, 2022



Test summary

17 0 0 0


Run details

Project Apiman
Status Passed
Commit b254f87 ℹ️
Started Jul 8, 2022 6:49 AM
Ended Jul 8, 2022 6:52 AM
Duration 03:23 💡
OS Linux Ubuntu - 20.04
Browser Electron 100

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@volkflo volkflo force-pushed the update-swagger-ui-to-v4 branch from 4cf7026 to 486d784 Compare May 31, 2022 06:17
Changed some methods according to the documentation.
The behaviour is now the same as in the apiman-developer-portal.
@volkflo volkflo force-pushed the update-swagger-ui-to-v4 branch from 486d784 to 2e0d3c1 Compare May 31, 2022 06:32
@volkflo volkflo marked this pull request as ready for review May 31, 2022 06:37
@volkflo
Copy link
Member Author

volkflo commented May 31, 2022

Seems to work fine now

@msavy
Copy link
Member

msavy commented Jul 4, 2022

Rebased onto latest deps

@msavy msavy changed the title build(deps): update swagger-ui to v4 refactor(ui): update swagger-ui to v4 Jul 7, 2022
@msavy
Copy link
Member

msavy commented Jul 7, 2022

image
image

Backgrounds of samples don't look right. Is this just a CSS thing?

@msavy
Copy link
Member

msavy commented Jul 7, 2022

Looks like it's caused by prism.js CSS (used also for markdown editor code section).

prism.css#L7

code[class*="language-"],
pre[class*="language-"] {
	color: black;  // <--- this one by the looks of it.
}

@msavy
Copy link
Member

msavy commented Jul 7, 2022

I'm not sure if it was a caching issue as I can't see the issue any more!

Copy link
Member

@msavy msavy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Putting on request changes, but might just be clarifications required.

manager/ui/war/package.json Outdated Show resolved Hide resolved
@volkflo volkflo requested a review from msavy July 8, 2022 06:36
@msavy msavy merged commit ad2e513 into master Jul 8, 2022
@msavy msavy deleted the update-swagger-ui-to-v4 branch July 8, 2022 07:51
Copy link
Member

@msavy msavy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good. Thanks!

@msavy msavy requested review from msavy and removed request for msavy July 8, 2022 07:52
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.

2 participants