-
Notifications
You must be signed in to change notification settings - Fork 2
Feature | Extend Swagger Coverage for controller OAuth2MembersApiController
#435
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
Feature | Extend Swagger Coverage for controller OAuth2MembersApiController
#435
Conversation
11f3972 to
3542180
Compare
|
@caseylocker the PR is ready to be reviewed |
f750c32 to
e01245e
Compare
caseylocker
left a comment
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.
LGTM. @smarcet if you agree please merge.
|
@matiasperrone-exo please review conflicts |
3ebecf4 to
2c501bf
Compare
c6ecdd0 to
728ae67
Compare
caseylocker
left a comment
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.
Approved. Mergeable.
…ity and request schemas
34b4c98 to
c4b8fb5
Compare
smarcet
left a comment
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.
LGTM
Tasks:
Ref: https://app.clickup.com/t/86b6wkhc6