-
Notifications
You must be signed in to change notification settings - Fork 155
[MM-47818] userInviteCmdF function to return an error in case of a failure #595
Conversation
Hello @ellisonleao, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. |
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.
Hi @ellisonleao, thank you very much for your contribution. I'd like to suggest a few improvements and the linter complains about the order of the imports.
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.
Thank you @ellisonleao, this looks good to me.
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.
Thanks for the contribution @ellisonleao LGTM!
Summary
Added multi error handing in userInviteCmdF function
Ticket Link
Fixes: mattermost/mattermost#21468
Jira: https://mattermost.atlassian.net/browse/MM-47818