Groups API getGroup and updateGroup aren't working for inactive groups with M2M

[rootelement]

These calls should allow an M2M with the right scopes to act just as an admin can.

groups-api/src/services/GroupService.js

Line 263 in 2711917

currentUser !== 'M2M' && helper.hasAdminRole(currentUser)

groups-api/src/services/GroupService.js

Line 323 in 2711917

const isAdmin = currentUser !== 'M2M' && helper.hasAdminRole(currentUser)

I believe the update is:
currentUser === 'M2M' || helper.hasAdminRole(currentUser)

But please confirm and test.

• Make sure you pass lint.
• Please confirm the swagger is up to date and valid (can be loaded into editor.swagger.io)
• Make sure any necessary postman updates are made
• If there are tests, please add/update tests for this
• Make a PR against develop.

cc @cwdcwd

[rootelement]

FYI - i don't think this has really ever been an issue for anyone else because I'm bringing legacy groups forward with a script and I was creating them as inactive, so i couldn't find them afterward.

[bountyC0d3r]

by design the group update was only done by the admin only and not by the m2m

[rootelement]

Ok, i'll conceed the Update part. But the GET won't allow an M2M to get Inactive groups?

[bountyC0d3r]

@rootelement I made changes where you can get the inactive groups in getGroups call