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

make webidentity/clientgrants response classes as inner-classes #1040

Conversation

balamurugana
Copy link
Member

No description provided.

@balamurugana balamurugana force-pushed the make-webidentity-clientgrants-response-classes-as-subclasses branch from 50d2d77 to c70e823 Compare August 17, 2020 16:33
@balamurugana balamurugana force-pushed the make-webidentity-clientgrants-response-classes-as-subclasses branch 3 times, most recently from e6c5d10 to 73e14cc Compare August 24, 2020 05:33
@balamurugana balamurugana changed the title make-webidentity/clientgrants response classes as subclasses make-webidentity/clientgrants response classes as inner-classes Aug 24, 2020
@balamurugana balamurugana changed the title make-webidentity/clientgrants response classes as inner-classes make webidentity/clientgrants response classes as inner-classes Aug 24, 2020
@balamurugana balamurugana force-pushed the make-webidentity-clientgrants-response-classes-as-subclasses branch from 73e14cc to 2413186 Compare August 24, 2020 05:37
Copy link
Contributor

@anjalshireesh anjalshireesh left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@sinhaashish sinhaashish left a comment

Choose a reason for hiding this comment

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

LGTM

@balamurugana balamurugana merged commit 1b6a8ca into minio:master Aug 24, 2020
@balamurugana balamurugana deleted the make-webidentity-clientgrants-response-classes-as-subclasses branch August 24, 2020 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants