Skip to content

fix: error parameter in GetUsers at default role manager#794

Merged
hsluoyz merged 1 commit intoapache:masterfrom
sagilio:master
May 24, 2021
Merged

fix: error parameter in GetUsers at default role manager#794
hsluoyz merged 1 commit intoapache:masterfrom
sagilio:master

Conversation

@sagilio
Copy link
Contributor

@sagilio sagilio commented May 17, 2021

No description provided.

Signed-off-by: Sagilio <Sagilio@outlook.com>
@hsluoyz hsluoyz requested a review from kilosonc May 17, 2021 16:16
@hsluoyz
Copy link
Member

hsluoyz commented May 17, 2021

@closetool

@hsluoyz
Copy link
Member

hsluoyz commented May 24, 2021

@closetool plz review.

@hsluoyz hsluoyz self-requested a review May 24, 2021 02:06
Copy link
Member

@hsluoyz hsluoyz left a comment

Choose a reason for hiding this comment

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

plz fix:

image

Copy link
Contributor

@kilosonc kilosonc left a comment

Choose a reason for hiding this comment

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

LGTM

@hsluoyz hsluoyz merged commit 63c66e3 into apache:master May 24, 2021
@github-actions
Copy link

🎉 This PR is included in version 2.30.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants