-
-
Notifications
You must be signed in to change notification settings - Fork 272
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
[17.0][IMP] base_user_role: Adding alert in user when role is used #296
Conversation
9b9c847
to
b7578b9
Compare
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
@rven could you review? Forward migration of a change in v16. |
@pedrobaeza could you also review this. This change was made on v16 after it has been migrated to v17, therefore this commit is missing in v17. |
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.
It's the same as v16:
/ocabot merge minor
On my way to merge this fine PR! |
Congratulations, your PR was merged at 214e386. Thanks a lot for contributing to OCA. ❤️ |
cherry pick of #273
@dreispt