-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: refactor with authorization, change current account type #1046
Conversation
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.
Looks good ! I have left a few comments.
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 🧑🚀 I wonder if we should make the optional parameters of PreventGuestWrapper
mandatory. What do you think (I elaborate that directly in the code review)?
de694e4
to
fd51a38
Compare
Quality Gate passedIssues Measures |
In this PR:
currentMember
typeWe can discuss the naming.