[MM-47825] refactor rolesSystemAdminCmdF function to return an error #21477

[yakuter]

Summary

Added multi error handing in rolesSystemAdminCmdF function

Ticket Link

Fixes: mattermost/mattermost#21477
JIRA: https://mattermost.atlassian.net/browse/MM-47825

[mattermod]

Hello @yakuter,

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.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

[yakuter]

/check-cla

[noxer]

Hi @yakuter, thank you very much for your contribution. I'd like to request a few minor changes, other than that it looks very good.

[yakuter]

@noxer thank you for the review. I solved and commited all the things you mentioned.

[noxer]

Just two minor adjustments, then we're done ;)
edit: One. Just one.

[noxer]

Perfect, thank you very much!

[yakuter]

Perfect, thank you very much!

Thanks to you @noxer :)

[isacikgoz]

Awesome, thanks for the contribution @yakuter 🎉

[yakuter]

Thanks for your support for my first PR to mattermost @noxer @isacikgoz 🥳