fix: add shareContent group permission and minor hub group clean up

[vivzhang]

1. Description:

• add new shareContent group permission policy
• add memberType to computedProps so it gets added to IHubGroup
• minor hub group clean up

1. Instructions for testing:

2. Closes Issues: # (if appropriate)

3. Updated meaningful TSDoc to methods including Parameters and Returns, see Documentation Guide

4. used semantic commit messages

5. PR title follows semantic commit format (CRITICAL if the title is not in a semantic format, the release automation will not run!)

6. updated peerDependencies as needed. CRITICAL our automated release system can not be counted on to update peerDependencies so we must do it manually in our PRs when needed. See the updating peerDependencies section of the release instructions for more details.

[dbouwman]

I think you also need to verify the user is a member of the group. We can do this with assertions...

assertions:[
  {  
      property: "context:currentUser",
      type: "is-group-member",
      value: "entity:id",
  }
]

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (ba14502) 100.00% compared to head (0d6ad6d) 100.00%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##            master     #1194   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          737       737           
  Lines        13163     13162    -1     
  Branches      2245      2245           
=========================================
- Hits         13163     13162    -1     

Files Changed	Coverage Δ
.../common/src/groups/_internal/GroupBusinessRules.ts	100.00% <ø> (ø)
packages/common/src/groups/HubGroup.ts	100.00% <100.00%> (ø)
...ckages/common/src/groups/_internal/computeProps.ts	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[juliannemarik]

You might want to add some protection here, e.g. getProp(group.userMembership, "memberType") or group.userMembership?.memberType. The API docs make it sound like userMembership is only returned on the response for authenticated users.

[juliannemarik]

Left one comment, otherwise this looks good to me