api/api/groups.py

@@ -24,7 +24,6 @@ def create(user, data):
         "name": data["name"],
         "description": data.get("description", ""),
         "closed": data.get("closed", False),
-        "memberPermissions": ["viewMembers", "viewNoticeboard", "postNoticeboard", "viewProjects", "postProjects"],
     }
     result = db.groups.insert_one(group)
     group["_id"] = result.inserted_id
@@ -44,10 +43,6 @@ def get_one(user, id):
     group = db.groups.find_one({"_id": id})
     if not group:
         raise util.errors.NotFound("Group not found")
-    if group.get("image"):
-        group["imageUrl"] = uploads.get_presigned_url(
-            "groups/{0}/{1}".format(id, group["image"])
-        )
     group["adminUsers"] = list(
         db.users.find(
             {"_id": {"$in": group.get("admins", [])}}, {"username": 1, "avatar": 1}
@@ -69,7 +64,7 @@ def update(user, id, update):
         raise util.errors.NotFound("Group not found")
     if user["_id"] not in group.get("admins", []):
         raise util.errors.Forbidden("You're not a group admin")
-    allowed_keys = ["name", "description", "closed", "memberPermissions", "image"]
+    allowed_keys = ["name", "description", "closed"]
     updater = util.build_updater(update, allowed_keys)
     if updater:
         db.groups.update_one({"_id": id}, updater)

api/app.py

@@ -478,8 +478,6 @@ def group_route(id):
         "name": fields.Str(),
         "description": fields.Str(),
         "closed": fields.Bool(),
-        "memberPermissions": fields.List(fields.Str()),
-        "image": fields.Str(allow_none=True),
     }
 )
 def group_route_put(args, id):

web/src/components/main/groups/Group.jsx

@@ -1,6 +1,6 @@
 import React, { useEffect } from 'react'
 import { Helmet } from 'react-helmet'
-import { Segment, Loader, Menu, Message, Container, Button, Icon, Grid, Card, Image } from 'semantic-ui-react'
+import { Segment, Loader, Menu, Message, Container, Button, Icon, Grid, Card } from 'semantic-ui-react'
 import { Outlet, Link, useParams } from 'react-router-dom'
 import { useDispatch, useSelector } from 'react-redux'
 import { toast } from 'react-toastify'
@@ -89,8 +89,6 @@ function Group () {
           <Grid stackable>
             <Grid.Column computer={4}>
               <Card fluid color='yellow'>
-                {group.imageUrl &&
-                  <Image src={group.imageUrl} wrapped ui={false} />}
                 {group.description &&
                   <Card.Content>{group.description}</Card.Content>}
                 {group.closed &&

web/src/components/main/groups/Settings.jsx

@@ -6,7 +6,6 @@ import { toast } from 'react-toastify'
 import utils from '../../../utils/utils.js'
 import actions from '../../../actions'
 import api from '../../../api'
-import FileChooser from '../../includes/FileChooser'
 
 const PERMISSIONS = [
   { name: 'viewMembers', label: 'Allow members to view other members' },
@@ -38,7 +37,6 @@ function Settings () {
       dispatch(actions.groups.request(false))
     })
   }
-
   const savePermission = (permissionName, enabled) => {
     const permissions = group.memberPermissions || []
     const index = permissions.indexOf(permissionName)
@@ -53,7 +51,6 @@ function Settings () {
       toast.error(err.message)
     })
   }
-
   const deleteGroup = () => {
     utils.confirm('Really delete this group?', 'You\'ll lose all entries in the group feed and anything else you\'ve added to it.').then(() => {
       api.groups.delete(group._id, () => {
@@ -64,16 +61,6 @@ function Settings () {
     }, () => {})
   }
 
-  const updatePicture = (image) => {
-    api.groups.update(group._id, { image }, g => {
-      if (!image) { // Needed to ensure the avatar is immediately unset
-        g.image = null
-        g.imageUrl = null
-      }
-      dispatch(actions.groups.updateGroup(group._id, { image: g.image, imageUrl: g.imageUrl }))
-    })
-  }
-
   return (
     <div>
       <Segment color='blue'>
@@ -89,25 +76,6 @@ function Settings () {
           <Divider hidden />
           <Form.Button loading={loading} color='teal' icon='check' content='Save changes' onClick={saveGroup} />
         </Form>
-
-        <Header>Group picture</Header>
-        <p>Upload a picture to represent your group. This will be shown on the group page and may be viewable to both members and non-members.</p>
-        {group.imageUrl &&
-          <div style={{ marginBottom: 10 }}>
-            <img src={group.imageUrl} alt='Group' style={{ maxWidth: 200, borderRadius: 5 }} />
-          </div>
-        }
-
-        <div style={{ display: 'flex', alignItems: 'center' }}>
-          {group.imageUrl &&
-            <Button basic color='gray' icon='times' content='Remove image' onClick={() => updatePicture(null)} />
-          }
-          <FileChooser
-            forType='group' forObject={group}
-            trigger={<Button basic color='yellow' icon='image' content='Choose an image' />}
-            accept='image/*' onComplete={f => updatePicture(f.storedName)}
-          />
-        </div>
       </Segment>
 
       <Segment color='blue'>