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.

Sign up for GitHub

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

Jump to bottom

Fix deprecation warnings for Swagger #549

Merged
merged 1 commit into from
Dec 13, 2023
Merged

Fix deprecation warnings for Swagger #549

merged 1 commit into from
Dec 13, 2023

Conversation

blacksam07
Copy link
Contributor

Board:

Description:

  • Replace deprecated methods for rswag gems

Notes:

17:10:57 web.1  | DEPRECATION WARNING: swagger_root= is deprecated and will be removed from rswag-api 3.0 (use openapi_root= instead) (called from block in <main> at /Users/samirtapiero/Backend/rails_api_base/config/initializers/rswag_api.rb:8)
17:10:57 web.1  | DEPRECATION WARNING: Rswag::Ui: WARNING: The method will be renamed to "openapi_endpoint" in v3.0 (called from block in <main> at /Users/samirtapiero/Backend/rails_api_base/config/initializers/rswag_ui.rb:12)

Preview:

image

santib
santib approved these changes Dec 12, 2023
View reviewed changes
Copy link
Member

@santib santib left a comment

Choose a reason for hiding this comment

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

👏 thanks for being proactive about this

@santib santib requested a review from a team December 12, 2023 22:48
@blacksam07 blacksam07 merged commit 2991e39 into main Dec 13, 2023
3 checks passed
@blacksam07 blacksam07 deleted the fix-deprecation-warnings-swagger branch December 13, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@santib santib santib approved these changes

@enzofab91 enzofab91 enzofab91 approved these changes

@juanmferreira93 juanmferreira93 juanmferreira93 approved these changes

@JulianPasquale JulianPasquale Awaiting requested review from JulianPasquale

@sebastiancaraballo sebastiancaraballo Awaiting requested review from sebastiancaraballo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@blacksam07 @santib @enzofab91 @juanmferreira93