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

chore(deps): upgrade swagger-ui-react to v5 #12540

Merged

Conversation

agilgur5
Copy link
Contributor

@agilgur5 agilgur5 commented Jan 17, 2024

Supersedes / Closes #12273
Follow-up to #11418 (comment), #11640 (comment), #12058

Motivation

v4 is no longer supported, so we should get onto the latest major, v5. v5 is also more performant, c.f. #12518 which updates Swagger UI for the docs and details that a bit more

  • with the current build system and the current version of swagger-ui-react, we can now update to v5 without errors
    • previously this had issues with older Webpack and older Babel, as well as loaded the Node bundle instead of the browser bundle (see the linked comments above)

NOTE: I did not update to v5.11.0 as that has a breaking change that is not compatible with our current version of React: swagger-api/swagger-ui#9495

Modifications

Upgrade swagger-ui-react from latest v4 to v5.10.0

Verification

Loaded the /apidocs page and confirmed it was working:
Screenshot 2024-01-17 at 6 31 50 PM

- v4 is no longer supported, so we should get onto the latest major, v5

- with the current build system and the current version of `swagger-ui-react`, we can now update to v5 without errors
  - previously this had issues with older Webpack and older Babel, as well as loaded the Node bundle instead of the browser bundle

Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
@agilgur5 agilgur5 added type/dependencies PRs and issues specific to updating dependencies area/ui javascript Pull requests that update Javascript dependencies labels Jan 17, 2024
@terrytangyuan terrytangyuan merged commit b734b66 into argoproj:main Jan 18, 2024
16 checks passed
@agilgur5 agilgur5 deleted the chore-deps-upgrade-swagger-ui-react-v5 branch January 18, 2024 16:22
isubasinghe pushed a commit to isubasinghe/argo-workflows that referenced this pull request Feb 27, 2024
isubasinghe pushed a commit to isubasinghe/argo-workflows that referenced this pull request Feb 27, 2024
isubasinghe pushed a commit to isubasinghe/argo-workflows that referenced this pull request Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui javascript Pull requests that update Javascript dependencies type/dependencies PRs and issues specific to updating dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants