From d9f07c1adbdf0b2ec0692da364e2b4736c9d4b6a Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 09:33:12 +0100
Subject: [PATCH 1/9] refactor widget footer, add possibility to delete project

---
 client/src/components/WidgetFooter.vue        |  11 +-
 .../components/portfolio/ProjectWidget.vue    |  43 ++++++-
 client/src/components/rooms/RoomWidget.vue    |  21 ++-
 client/src/components/rooms/WidgetPopover.vue |  33 +++--
 .../graphql/gql/mutations/deleteProject.gql   |   7 +
 server/portfolio/factories.py                 |  17 +++
 .../migrations/0003_auto_20190325_1452.py     |  27 ++++
 server/portfolio/models.py                    |   3 +
 server/portfolio/mutations.py                 |  36 +++++-
 server/portfolio/schema.py                    |  10 +-
 server/portfolio/serializers.py               |   2 +-
 server/portfolio/tests/__init__.py            |  10 ++
 .../portfolio/tests/test_project_mutations.py |  62 +++++++++
 server/portfolio/tests/test_project_query.py  | 120 ++++++++++++++++++
 .../test_room_delete_edit_permissions.py      |   1 -
 server/users/services.py                      |   3 -
 16 files changed, 363 insertions(+), 43 deletions(-)
 create mode 100644 client/src/graphql/gql/mutations/deleteProject.gql
 create mode 100644 server/portfolio/factories.py
 create mode 100644 server/portfolio/migrations/0003_auto_20190325_1452.py
 create mode 100644 server/portfolio/tests/__init__.py
 create mode 100644 server/portfolio/tests/test_project_mutations.py
 create mode 100644 server/portfolio/tests/test_project_query.py

diff --git a/client/src/components/WidgetFooter.vue b/client/src/components/WidgetFooter.vue
index 9ec6e1a5..cc06f221 100644
--- a/client/src/components/WidgetFooter.vue
+++ b/client/src/components/WidgetFooter.vue
@@ -3,12 +3,10 @@
     <a @click="showMenu = !showMenu" class="widget-footer__more-link">
       <ellipses></ellipses>
     </a>
-    <widget-popover :entity="entity"
-                    @delete="onDelete"
-                    @hide-me="showMenu = false"
-                    @edit="onEdit"
-                    :id="id"
-                    v-if="showMenu"></widget-popover>
+    <widget-popover v-if="showMenu"
+                    @hide-me="showMenu = false">
+      <slot></slot>
+    </widget-popover>
   </div>
 </template>
 
@@ -17,7 +15,6 @@
   import WidgetPopover from '@/components/rooms/WidgetPopover';
 
   export default {
-    props: ['on-delete', 'on-edit', 'id', 'entity'],
 
     components: {
       Ellipses,
diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index e96bd94e..5e9d277b 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -7,19 +7,25 @@
       <owner-widget name="Hans Muster"></owner-widget>
 
     </router-link>
-    <widget-footer
-      entity="Eintrag"
-    ></widget-footer>
+    <widget-footer>
+      <li class="popover-links__link"><a @click="deleteProject()">Projekt löschen</a></li>
+      <li class="popover-links__link"><a @click="editProject()">Projekt bearbeiten</a></li>
+      <li class="popover-links__link"><a @click="shareProject()">Projekt teilen</a></li>
+    </widget-footer>
   </div>
 </template>
 
 <script>
+
+  import DELETE_PROJECT_MUTATION from '@/graphql/gql/mutations/deleteProject.gql';
+  import PROJECTS_QUERY from '@/graphql/gql/allProjects.gql';
+
   import OwnerWidget from '@/components/portfolio/OwnerWidget';
   import EntryCountWidget from '@/components/rooms/EntryCountWidget';
   import WidgetFooter from '@/components/WidgetFooter';
 
   export default {
-    props: ['title', 'appearance', 'slug'],
+    props: ['title', 'appearance', 'slug', 'id'],
 
     components: {
       WidgetFooter,
@@ -31,6 +37,35 @@
       widgetClass() {
         return `project-widget--${this.appearance}`;
       }
+    },
+    methods: {
+      deleteProject() {
+        const theId = this.id
+        this.$apollo.mutate({
+          mutation: DELETE_PROJECT_MUTATION,
+          variables: {
+            input: {
+              id: theId
+            }
+          },
+          update(store, {data: {deleteProject: {success}}}) {
+            if (success) {
+              const data = store.readQuery({query: PROJECTS_QUERY});
+
+              if (data) {
+                data.projects.edges.splice(data.projects.edges.findIndex(edge => edge.node.id === theId), 1);
+                store.writeQuery({query: PROJECTS_QUERY, data});
+              }
+            }
+          }
+        })
+      },
+      editProject() {
+        this.$router.push({name: 'edit-room', params: {id: this.id}});
+      },
+      shareProject() {
+
+      }
     }
   }
 </script>
diff --git a/client/src/components/rooms/RoomWidget.vue b/client/src/components/rooms/RoomWidget.vue
index e0562355..aff1f758 100644
--- a/client/src/components/rooms/RoomWidget.vue
+++ b/client/src/components/rooms/RoomWidget.vue
@@ -5,13 +5,9 @@
       <room-group-widget v-bind="schoolClass"></room-group-widget>
       <entry-count-widget :entryCount="entryCount"></entry-count-widget>
     </router-link>
-    <widget-footer
-      v-if="canEditRoom"
-      :on-delete="deleteRoom"
-      :on-edit="editRoom"
-      :id="id"
-      entity="Raum"
-    >
+    <widget-footer v-if="canEditRoom">
+      <li class="popover-links__link"><a @click="deleteRoom()">Raum löschen</a></li>
+      <li class="popover-links__link"><a @click="editRoom()">Raum bearbeiten</a></li>
     </widget-footer>
   </div>
 </template>
@@ -58,27 +54,28 @@
     },
 
     methods: {
-      deleteRoom(id) {
+      deleteRoom() {
+        const theId = this.id
         this.$apollo.mutate({
           mutation: DELETE_ROOM_MUTATION,
           variables: {
             input: {
-              id
+              id: theId
             }
           },
           update(store, {data: {deleteRoom: {success}}}) {
             if (success) {
               const data = store.readQuery({query: ROOMS_QUERY});
               if (data) {
-                data.rooms.edges.splice(data.rooms.edges.findIndex(edge => edge.node.id === id), 1);
+                data.rooms.edges.splice(data.rooms.edges.findIndex(edge => edge.node.id === theId), 1);
                 store.writeQuery({query: ROOMS_QUERY, data});
               }
             }
           }
         })
       },
-      editRoom(id) {
-        this.$router.push({name: 'edit-room', params: {id: id}});
+      editRoom() {
+        this.$router.push({name: 'edit-room', params: {id: this.id}});
       }
     }
   }
diff --git a/client/src/components/rooms/WidgetPopover.vue b/client/src/components/rooms/WidgetPopover.vue
index 45d53c24..e5a8f8c3 100644
--- a/client/src/components/rooms/WidgetPopover.vue
+++ b/client/src/components/rooms/WidgetPopover.vue
@@ -1,14 +1,13 @@
 <template>
-  <div class="room-popover" v-click-outside="hidePopover">
-    <a class="room-popover__link" @click="$emit('delete', id)">{{entity}} löschen</a>
-    <a class="room-popover__link" @click="$emit('edit', id)">{{entity}} bearbeiten</a>
+  <div class="widget-popover" v-click-outside="hidePopover">
+    <ul class="widget-popover__links popover-links">
+        <slot></slot>
+    </ul>
   </div>
 </template>
 
 <script>
   export default {
-    props: ['id', 'entity'],
-
     methods: {
       hidePopover() {
         this.$emit('hide-me');
@@ -21,23 +20,31 @@
   @import "@/styles/_variables.scss";
   @import "@/styles/_mixins.scss";
 
-  .room-popover {
+  .widget-popover {
     position: absolute;
     right: 0;
     bottom: -110px;
-    display: grid;
     background-color: $color-white;
     padding: 20px;
     z-index: 10;
     @include widget-shadow;
+  }
+
+  .popover-links {
+
+    list-style: none;
+    display: grid;
 
     &__link {
-      color: $color-grey;
-      font-family: $sans-serif-font-family;
-      font-size: toRem(14px);
-      line-height: 1.5;
-      padding: 5px 0;
-      cursor: pointer;
+      & > a {
+        display: inline-block;
+        color: $color-grey;
+        font-family: $sans-serif-font-family;
+        font-size: toRem(14px);
+        line-height: 1.5;
+        padding: 5px 0;
+        cursor: pointer;
+      }
     }
   }
 </style>
diff --git a/client/src/graphql/gql/mutations/deleteProject.gql b/client/src/graphql/gql/mutations/deleteProject.gql
new file mode 100644
index 00000000..0a2e671e
--- /dev/null
+++ b/client/src/graphql/gql/mutations/deleteProject.gql
@@ -0,0 +1,7 @@
+mutation DeleteProject($input: DeleteProjectInput!) {
+  deleteProject(input: $input) {
+    success
+    errors
+  }
+}
+
diff --git a/server/portfolio/factories.py b/server/portfolio/factories.py
new file mode 100644
index 00000000..4432b5fc
--- /dev/null
+++ b/server/portfolio/factories.py
@@ -0,0 +1,17 @@
+import random
+
+import factory
+
+from core.factories import fake
+from portfolio.models import Project, ProjectEntry
+
+
+class ProjectFactory(factory.django.DjangoModelFactory):
+    class Meta:
+        model = Project
+
+    objectives = factory.LazyAttribute(lambda x: fake.sentence(nb_words=random.randint(4, 8)))
+    title = factory.LazyAttribute(lambda x: fake.sentence(nb_words=random.randint(4, 8)))
+    appearance = factory.LazyAttribute(lambda x: random.choice(['red', 'green', 'yellow']))
+    final = False
+
diff --git a/server/portfolio/migrations/0003_auto_20190325_1452.py b/server/portfolio/migrations/0003_auto_20190325_1452.py
new file mode 100644
index 00000000..a04c9f33
--- /dev/null
+++ b/server/portfolio/migrations/0003_auto_20190325_1452.py
@@ -0,0 +1,27 @@
+# Generated by Django 2.0.6 on 2019-03-25 14:52
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('portfolio', '0002_projectentry'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='project',
+            name='final',
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name='project',
+            name='student',
+            field=models.ForeignKey(default=None, on_delete=django.db.models.deletion.CASCADE, related_name='projects', to=settings.AUTH_USER_MODEL),
+            preserve_default=False,
+        ),
+    ]
diff --git a/server/portfolio/models.py b/server/portfolio/models.py
index 32ca421b..9a601463 100644
--- a/server/portfolio/models.py
+++ b/server/portfolio/models.py
@@ -1,3 +1,4 @@
+from django.contrib.auth import get_user_model
 from django.db import models
 from django_extensions.db.models import TitleSlugDescriptionModel
 
@@ -5,6 +6,8 @@ from django_extensions.db.models import TitleSlugDescriptionModel
 class Project(TitleSlugDescriptionModel):
     objectives = models.TextField(blank=True)
     appearance = models.CharField(blank=True, null=False, max_length=255)
+    student = models.ForeignKey(get_user_model(), on_delete=models.CASCADE, related_name='projects')
+    final = models.BooleanField(default=False)
 
     def __str__(self):
         return self.title
diff --git a/server/portfolio/mutations.py b/server/portfolio/mutations.py
index bbb6ebc6..985c6302 100644
--- a/server/portfolio/mutations.py
+++ b/server/portfolio/mutations.py
@@ -1,5 +1,6 @@
 import graphene
 from graphene import relay
+from rest_framework.exceptions import PermissionDenied
 
 from api.utils import get_object
 from portfolio.inputs import AddProjectArgument, UpdateProjectArgument, AddProjectEntryArgument, \
@@ -31,7 +32,6 @@ from portfolio.serializers import ProjectSerializer, ProjectEntrySerializer
 #
 #         return cls(errors=['{}: {}'.format(key, value) for key, value in serializer.errors.items()])
 
-
 class MutateProject(relay.ClientIDMutation):
     errors = graphene.List(graphene.String)
     project = graphene.Field(ProjectNode)
@@ -64,6 +64,19 @@ class AddProject(MutateProject):
         project = graphene.Argument(
             AddProjectArgument)  # NB: can't be named AddProjectInput, otherwise graphene complains
 
+    @classmethod
+    def mutate_and_get_payload(cls, root, info, **kwargs):
+
+        data = kwargs.get('project')
+        data['student'] = info.context.user.id
+
+        serializer = ProjectSerializer(data=data)
+        if serializer.is_valid():
+            serializer.save()
+            return cls(project=serializer.instance)
+
+        return cls(room=None, errors=['{}: {}'.format(key, value) for key, value in serializer.errors.items()])
+
 
 class UpdateProject(MutateProject):
     class Input:
@@ -102,8 +115,29 @@ class UpdateProjectEntry(MutateProjectEntry):
         project_entry = graphene.Argument(UpdateProjectEntryArgument)
 
 
+class DeleteProject(relay.ClientIDMutation):
+    class Input:
+        id = graphene.ID(required=True)
+
+    success = graphene.Boolean()
+    errors = graphene.List(graphene.String)
+
+    @classmethod
+    def mutate_and_get_payload(cls, root, info, **kwargs):
+        id = kwargs.get('id')
+        project = get_object(Project, id)
+        user = info.context.user
+
+        if project.student != user:
+            raise PermissionDenied('Permission denied: Incorrect project')
+        project.delete()
+        return cls(success=True)
+
+
 class PortfolioMutations:
     add_project = AddProject.Field()
     update_project = UpdateProject.Field()
+    delete_project = DeleteProject.Field()
     add_project_entry = AddProjectEntry.Field()
     update_project_entry = UpdateProjectEntry.Field()
+
diff --git a/server/portfolio/schema.py b/server/portfolio/schema.py
index 2edf9f91..9e7b9864 100644
--- a/server/portfolio/schema.py
+++ b/server/portfolio/schema.py
@@ -5,6 +5,7 @@ from graphene_django.filter import DjangoFilterConnectionField
 
 from api.utils import get_by_id_or_slug
 from portfolio.models import Project, ProjectEntry
+from users.models import UserRole, Role
 
 
 class ProjectNode(DjangoObjectType):
@@ -30,7 +31,14 @@ class PortfolioQuery(object):
     projects = DjangoFilterConnectionField(ProjectNode)
 
     def resolve_projects(self, info, **kwargs):
-        return Project.objects.all().order_by('-pk')
+        user = info.context.user
+        if user.is_superuser:
+            return Project.objects.all().order_by('-pk')
+
+        if UserRole.get_role_for_user(user).role == Role.objects.get_default_teacher_role():
+            return Project.objects.filter(student__school_classes__in=user.school_classes.all(), final=True)
+
+        return Project.objects.filter(student=user)
 
     def resolve_project(self, info, **kwargs):
         return get_by_id_or_slug(Project, **kwargs)
diff --git a/server/portfolio/serializers.py b/server/portfolio/serializers.py
index ed186aa8..939a9f98 100644
--- a/server/portfolio/serializers.py
+++ b/server/portfolio/serializers.py
@@ -6,7 +6,7 @@ from portfolio.models import Project, ProjectEntry
 class ProjectSerializer(serializers.ModelSerializer):
     class Meta:
         model = Project
-        fields = ('id', 'title', 'description', 'objectives', 'slug', 'appearance',)
+        fields = ('id', 'title', 'description', 'objectives', 'slug', 'appearance', 'student',)
         read_only_fields = ('id', 'slug',)
 
 
diff --git a/server/portfolio/tests/__init__.py b/server/portfolio/tests/__init__.py
new file mode 100644
index 00000000..336672d4
--- /dev/null
+++ b/server/portfolio/tests/__init__.py
@@ -0,0 +1,10 @@
+# -*- coding: utf-8 -*-
+#
+# ITerativ GmbH
+# http://www.iterativ.ch/
+#
+# Copyright (c) 2019 ITerativ GmbH. All rights reserved.
+#
+# Created on 26.03.19
+# @author: chrigu <christian.cueni@iterativ.ch>
+from django.conf import settings
diff --git a/server/portfolio/tests/test_project_mutations.py b/server/portfolio/tests/test_project_mutations.py
new file mode 100644
index 00000000..d44324ab
--- /dev/null
+++ b/server/portfolio/tests/test_project_mutations.py
@@ -0,0 +1,62 @@
+from django.test import TestCase, RequestFactory
+from graphene.test import Client
+from graphql_relay import to_global_id
+
+from api.schema import schema
+from portfolio.factories import ProjectFactory
+from portfolio.models import Project
+from rooms.models import Room
+from users.factories import SchoolClassFactory
+from users.models import User, SchoolClass
+from users.services import create_users
+
+
+class ProjectQuery(TestCase):
+    def setUp(self):
+        create_users()
+        self.teacher = User.objects.get(username='teacher')
+        self.teacher2 = User.objects.get(username='teacher2')
+        self.student = User.objects.get(username='student1')
+        self.student2 = User.objects.get(username='student2')
+        school_class1 = SchoolClassFactory(users=[self.teacher, self.student])
+        school_class2 = SchoolClassFactory(users=[self.teacher2, self.student2])
+        self.project1 = ProjectFactory(student=self.student)
+
+        self.mutation = '''
+        mutation DeleteProject($input: DeleteProjectInput!) {
+          deleteProject(input: $input) {
+            success
+            errors
+          }
+        }
+        '''
+
+        self.variables = {
+            'input': {
+                'id': to_global_id('ProjectNode', self.project1.id)
+            }
+        }
+
+    def test_should_be_able_to_delete_own_projects(self):
+
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.student
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.mutation, variables=self.variables)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(Project.objects.count(), 0)
+
+    def test_should_not_be_able_to_delete_other_projects(self):
+
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.student2
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.mutation, variables=self.variables)
+        self.assertEqual(result.get('errors')[0]['message'], 'Permission denied: Incorrect project')
+
+
diff --git a/server/portfolio/tests/test_project_query.py b/server/portfolio/tests/test_project_query.py
new file mode 100644
index 00000000..4ff1d7a4
--- /dev/null
+++ b/server/portfolio/tests/test_project_query.py
@@ -0,0 +1,120 @@
+from django.test import TestCase, RequestFactory
+from graphene.test import Client
+from graphql_relay import to_global_id
+
+from api.schema import schema
+from portfolio.factories import ProjectFactory
+from portfolio.models import Project
+from rooms.models import Room
+from users.factories import SchoolClassFactory
+from users.models import User, SchoolClass
+from users.services import create_users
+
+
+class ProjectQuery(TestCase):
+    def setUp(self):
+        create_users()
+        self.teacher = User.objects.get(username='teacher')
+        self.teacher2 = User.objects.get(username='teacher2')
+        self.student = User.objects.get(username='student1')
+        self.student2 = User.objects.get(username='student2')
+        school_class1 = SchoolClassFactory(users=[self.teacher, self.student])
+        school_class2 = SchoolClassFactory(users=[self.teacher2, self.student2])
+        self.project1 = ProjectFactory(student=self.student)
+        self.query = '''
+            query ProjectsQuery {
+              projects {
+                edges {
+                  node {
+                    ...ProjectParts
+                    __typename
+                  }
+                  __typename
+                }
+                __typename
+              }
+            }
+            
+            fragment ProjectParts on ProjectNode {
+              id
+              title
+              appearance
+              description
+              slug
+              objectives
+              __typename
+            }
+            
+        '''
+
+    def test_should_see_own_projects(self):
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.student
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(result.get('data').get('projects').get('edges')[0].get('node').get('title'), self.project1.title)
+
+    def test_should_not_see_other_projects(self):
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.student2
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(len(result.get('data').get('projects').get('edges')), 0)
+
+    def test_teacher_should_not_see_unfinished_projects(self):
+        request = RequestFactory().get('/')
+        request.user = self.teacher
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(len(result.get('data').get('projects').get('edges')), 0)
+
+    def test_teacher_should_only_see_finished_projects(self):
+        self.project1.final = True
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.teacher
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(result.get('data').get('projects').get('edges')[0].get('node').get('title'),
+                         self.project1.title)
+
+    def test_teacher_should_only_see_finished_projects(self):
+        self.project1.final = True
+        self.project1.save()
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.teacher
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(result.get('data').get('projects').get('edges')[0].get('node').get('title'),
+                         self.project1.title)
+
+    def test_other_teacher_should_not_see_projects(self):
+        self.project1.final = True
+        self.project1.save()
+        self.assertEqual(Project.objects.count(), 1)
+        request = RequestFactory().get('/')
+        request.user = self.teacher2
+        self.client = Client(schema=schema, context_value=request)
+
+        result = self.client.execute(self.query)
+
+        self.assertIsNone(result.get('errors'))
+        self.assertEqual(len(result.get('data').get('projects').get('edges')), 0)
diff --git a/server/rooms/tests/test_room_delete_edit_permissions.py b/server/rooms/tests/test_room_delete_edit_permissions.py
index 9ef3833d..b0ea2bd5 100644
--- a/server/rooms/tests/test_room_delete_edit_permissions.py
+++ b/server/rooms/tests/test_room_delete_edit_permissions.py
@@ -37,7 +37,6 @@ class RoomDeleteEditPermissionsTestcase(TestCase):
         request.user = self.teacher
         self.client = Client(schema=schema, context_value=request)
 
-
         result = self.client.execute(self.mutation, variables=self.variables)
 
         self.assertIsNone(result.get('errors'))
diff --git a/server/users/services.py b/server/users/services.py
index 60b796fc..b075cc2d 100644
--- a/server/users/services.py
+++ b/server/users/services.py
@@ -34,9 +34,6 @@ def create_users(data=None):
             name='second_class'
         )
 
-
-
-
     else:
         for school_class in data:
             first, last = school_class.get('teacher')

From 2b39cc92dd1dd6a6705b59d6b60d6065e2c229ac Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 14:09:34 +0100
Subject: [PATCH 2/9] implement share project button

---
 client/src/components/WidgetFooter.vue        | 10 +++-
 .../components/portfolio/ProjectWidget.vue    | 42 ++++-----------
 client/src/pages/portfolio.vue                | 51 ++++++++++++++++++-
 server/portfolio/inputs.py                    |  2 +
 server/portfolio/mutations.py                 |  4 +-
 server/portfolio/serializers.py               |  2 +-
 6 files changed, 73 insertions(+), 38 deletions(-)

diff --git a/client/src/components/WidgetFooter.vue b/client/src/components/WidgetFooter.vue
index cc06f221..1de28b7c 100644
--- a/client/src/components/WidgetFooter.vue
+++ b/client/src/components/WidgetFooter.vue
@@ -1,11 +1,11 @@
 <template>
   <div class="widget-footer">
-    <a @click="showMenu = !showMenu" class="widget-footer__more-link">
+    <a @click="toggleMenu" class="widget-footer__more-link">
       <ellipses></ellipses>
     </a>
     <widget-popover v-if="showMenu"
                     @hide-me="showMenu = false">
-      <slot></slot>
+      <slot :hide="toggleMenu"></slot>
     </widget-popover>
   </div>
 </template>
@@ -25,6 +25,12 @@
       return {
         showMenu: false
       }
+    },
+
+    methods: {
+      toggleMenu: function () {
+        this.showMenu = !this.showMenu
+      }
     }
   }
 </script>
diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index 5e9d277b..a9317657 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -8,18 +8,16 @@
 
     </router-link>
     <widget-footer>
-      <li class="popover-links__link"><a @click="deleteProject()">Projekt löschen</a></li>
-      <li class="popover-links__link"><a @click="editProject()">Projekt bearbeiten</a></li>
-      <li class="popover-links__link"><a @click="shareProject()">Projekt teilen</a></li>
+      <template slot-scope="scope">
+          <li class="popover-links__link"><a @click="$emit('delete', id)">Projekt löschen</a></li>
+          <li class="popover-links__link"><a @click="$emit('edit', id)">Projekt bearbeiten</a></li>
+          <li class="popover-links__link"><a @click="share(scope)">Projekt teilen</a></li>
+      </template>
     </widget-footer>
   </div>
 </template>
 
 <script>
-
-  import DELETE_PROJECT_MUTATION from '@/graphql/gql/mutations/deleteProject.gql';
-  import PROJECTS_QUERY from '@/graphql/gql/allProjects.gql';
-
   import OwnerWidget from '@/components/portfolio/OwnerWidget';
   import EntryCountWidget from '@/components/rooms/EntryCountWidget';
   import WidgetFooter from '@/components/WidgetFooter';
@@ -38,33 +36,11 @@
         return `project-widget--${this.appearance}`;
       }
     },
+
     methods: {
-      deleteProject() {
-        const theId = this.id
-        this.$apollo.mutate({
-          mutation: DELETE_PROJECT_MUTATION,
-          variables: {
-            input: {
-              id: theId
-            }
-          },
-          update(store, {data: {deleteProject: {success}}}) {
-            if (success) {
-              const data = store.readQuery({query: PROJECTS_QUERY});
-
-              if (data) {
-                data.projects.edges.splice(data.projects.edges.findIndex(edge => edge.node.id === theId), 1);
-                store.writeQuery({query: PROJECTS_QUERY, data});
-              }
-            }
-          }
-        })
-      },
-      editProject() {
-        this.$router.push({name: 'edit-room', params: {id: this.id}});
-      },
-      shareProject() {
-
+      share: function (scope) {
+        scope.hide();
+        this.$emit('share', this.id);
       }
     }
   }
diff --git a/client/src/pages/portfolio.vue b/client/src/pages/portfolio.vue
index a66cf482..b156f63a 100644
--- a/client/src/pages/portfolio.vue
+++ b/client/src/pages/portfolio.vue
@@ -5,7 +5,11 @@
 
       <project-widget
         v-for="project in projects"
-        v-bind="project" :key="project.id"
+        v-bind="project"
+        @delete="deleteProject"
+        @share="shareProject"
+        @edit="editProject"
+        :key="project.id"
         class="portfolio__project"
       ></project-widget>
     </div>
@@ -18,6 +22,8 @@
   import AddProject from '@/components/portfolio/AddProject';
 
   import PROJECTS_QUERY from '@/graphql/gql/allProjects.gql';
+  import DELETE_PROJECT_MUTATION from '@/graphql/gql/mutations/deleteProject.gql';
+  import UPDATE_PROJECT_MUTATION from '@/graphql/gql/mutations/updateProject.gql';
 
   export default {
     components: {
@@ -38,6 +44,49 @@
       return {
         projects: []
       }
+    },
+    methods: {
+      deleteProject(id) {
+        this.$apollo.mutate({
+          mutation: DELETE_PROJECT_MUTATION,
+          variables: {
+            input: {
+              id
+            }
+          },
+          update(store, {data: {deleteProject: {success}}}) {
+            if (success) {
+              const data = store.readQuery({query: PROJECTS_QUERY});
+
+              if (data) {
+                data.projects.edges.splice(data.projects.edges.findIndex(edge => edge.node.id === id), 1);
+                store.writeQuery({query: PROJECTS_QUERY, data});
+              }
+            }
+          }
+        })
+      },
+      editProject(id) {
+        this.$router.push({name: 'edit-project', params: { id }});
+      },
+      shareProject(id) {
+        const project = this.projects.filter(project => project.id === id)[0];
+        this.$apollo.mutate({
+          mutation: UPDATE_PROJECT_MUTATION,
+          variables: {
+            input: {
+              project: {
+                id: project.id,
+                title: project.title,
+                description: project.description,
+                appearance: project.appearance,
+                objectives: project.objectives,
+                final: true
+              }
+            }
+          }
+        })
+      }
     }
   }
 </script>
diff --git a/server/portfolio/inputs.py b/server/portfolio/inputs.py
index 584b8018..ecfda7a3 100644
--- a/server/portfolio/inputs.py
+++ b/server/portfolio/inputs.py
@@ -15,6 +15,7 @@ class AddProjectArgument(ProjectInput):
 
 class UpdateProjectArgument(ProjectInput):
     id = graphene.ID(required=True)
+    final = graphene.Boolean()
 
 
 class ProjectEntryInput(InputObjectType):
@@ -29,3 +30,4 @@ class AddProjectEntryArgument(ProjectEntryInput):
 
 class UpdateProjectEntryArgument(ProjectEntryInput):
     id = graphene.ID(required=True)
+
diff --git a/server/portfolio/mutations.py b/server/portfolio/mutations.py
index 985c6302..8e6dee3f 100644
--- a/server/portfolio/mutations.py
+++ b/server/portfolio/mutations.py
@@ -41,8 +41,10 @@ class MutateProject(relay.ClientIDMutation):
     #     serializer_class = ProjectSerializer
 
     @classmethod
-    def mutate_and_get_payload(cls, *args, **kwargs):
+    def mutate_and_get_payload(cls, root, info, **kwargs):
         data = kwargs.get('project')
+        data['student'] = info.context.user.id
+
         if data.get('id') is not None:
             entity = get_object(Project, data['id'])
             serializer = ProjectSerializer(entity, data=data)
diff --git a/server/portfolio/serializers.py b/server/portfolio/serializers.py
index 939a9f98..dc19d3b3 100644
--- a/server/portfolio/serializers.py
+++ b/server/portfolio/serializers.py
@@ -6,7 +6,7 @@ from portfolio.models import Project, ProjectEntry
 class ProjectSerializer(serializers.ModelSerializer):
     class Meta:
         model = Project
-        fields = ('id', 'title', 'description', 'objectives', 'slug', 'appearance', 'student',)
+        fields = ('id', 'title', 'description', 'objectives', 'slug', 'appearance', 'student', 'final',)
         read_only_fields = ('id', 'slug',)
 
 

From 0e1bca4c787c55ff57599a9c636b2f40bb8e0e47 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 14:20:21 +0100
Subject: [PATCH 3/9] add unshare for projects

---
 client/src/components/portfolio/ProjectWidget.vue | 14 ++++++++++----
 client/src/graphql/gql/fragments/projectParts.gql |  1 +
 client/src/pages/portfolio.vue                    |  6 +++---
 3 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index a9317657..9e54ec83 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -5,13 +5,13 @@
 
       <entry-count-widget entry-count="4"></entry-count-widget>
       <owner-widget name="Hans Muster"></owner-widget>
-
     </router-link>
     <widget-footer>
       <template slot-scope="scope">
           <li class="popover-links__link"><a @click="$emit('delete', id)">Projekt löschen</a></li>
           <li class="popover-links__link"><a @click="$emit('edit', id)">Projekt bearbeiten</a></li>
-          <li class="popover-links__link"><a @click="share(scope)">Projekt teilen</a></li>
+          <li v-if="!final" class="popover-links__link"><a @click="share(scope)">Projekt teilen</a></li>
+          <li v-if="final" class="popover-links__link"><a @click="unshare(scope)">Projekt nicht mehr teilen</a></li>
       </template>
     </widget-footer>
   </div>
@@ -23,7 +23,7 @@
   import WidgetFooter from '@/components/WidgetFooter';
 
   export default {
-    props: ['title', 'appearance', 'slug', 'id'],
+    props: ['title', 'appearance', 'slug', 'id', 'final'],
 
     components: {
       WidgetFooter,
@@ -39,8 +39,14 @@
 
     methods: {
       share: function (scope) {
+        this.updateShare(scope, true);
+      },
+      unshare: function (scope) {
+        this.updateShare(scope, false);
+      },
+      updateShare: function (scope, state) {
         scope.hide();
-        this.$emit('share', this.id);
+        this.$emit('updateShare', this.id, state);
       }
     }
   }
diff --git a/client/src/graphql/gql/fragments/projectParts.gql b/client/src/graphql/gql/fragments/projectParts.gql
index 04cf7c9c..c0d1a8b5 100644
--- a/client/src/graphql/gql/fragments/projectParts.gql
+++ b/client/src/graphql/gql/fragments/projectParts.gql
@@ -5,4 +5,5 @@ fragment ProjectParts on ProjectNode {
   description
   slug
   objectives
+  final
 }
diff --git a/client/src/pages/portfolio.vue b/client/src/pages/portfolio.vue
index b156f63a..cb5425cb 100644
--- a/client/src/pages/portfolio.vue
+++ b/client/src/pages/portfolio.vue
@@ -7,7 +7,7 @@
         v-for="project in projects"
         v-bind="project"
         @delete="deleteProject"
-        @share="shareProject"
+        @updateShare="updateShareState"
         @edit="editProject"
         :key="project.id"
         class="portfolio__project"
@@ -69,7 +69,7 @@
       editProject(id) {
         this.$router.push({name: 'edit-project', params: { id }});
       },
-      shareProject(id) {
+      updateShareState(id, state) {
         const project = this.projects.filter(project => project.id === id)[0];
         this.$apollo.mutate({
           mutation: UPDATE_PROJECT_MUTATION,
@@ -81,7 +81,7 @@
                 description: project.description,
                 appearance: project.appearance,
                 objectives: project.objectives,
-                final: true
+                final: state
               }
             }
           }

From cdd31516cf90dc6c38ded28d3fc4366ad3be2495 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 15:26:12 +0100
Subject: [PATCH 4/9] hide portfolio menu on role condition

---
 client/src/components/WidgetFooter.vue        |  3 ++-
 .../components/portfolio/ProjectWidget.vue    |  4 ++--
 client/src/pages/portfolio.vue                | 24 ++++++++++++++-----
 3 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/client/src/components/WidgetFooter.vue b/client/src/components/WidgetFooter.vue
index 1de28b7c..ec3db215 100644
--- a/client/src/components/WidgetFooter.vue
+++ b/client/src/components/WidgetFooter.vue
@@ -1,6 +1,7 @@
 <template>
   <div class="widget-footer">
-    <a @click="toggleMenu" class="widget-footer__more-link">
+    <a @click="toggleMenu"
+        class="widget-footer__more-link">
       <ellipses></ellipses>
     </a>
     <widget-popover v-if="showMenu"
diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index 9e54ec83..2ac339ec 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -6,7 +6,7 @@
       <entry-count-widget entry-count="4"></entry-count-widget>
       <owner-widget name="Hans Muster"></owner-widget>
     </router-link>
-    <widget-footer>
+    <widget-footer v-if="!isTeacher">
       <template slot-scope="scope">
           <li class="popover-links__link"><a @click="$emit('delete', id)">Projekt löschen</a></li>
           <li class="popover-links__link"><a @click="$emit('edit', id)">Projekt bearbeiten</a></li>
@@ -23,7 +23,7 @@
   import WidgetFooter from '@/components/WidgetFooter';
 
   export default {
-    props: ['title', 'appearance', 'slug', 'id', 'final'],
+    props: ['title', 'appearance', 'slug', 'id', 'final', 'isTeacher'],
 
     components: {
       WidgetFooter,
diff --git a/client/src/pages/portfolio.vue b/client/src/pages/portfolio.vue
index cb5425cb..2197a961 100644
--- a/client/src/pages/portfolio.vue
+++ b/client/src/pages/portfolio.vue
@@ -6,6 +6,7 @@
       <project-widget
         v-for="project in projects"
         v-bind="project"
+        :isTeacher="canManageContent"
         @delete="deleteProject"
         @updateShare="updateShareState"
         @edit="editProject"
@@ -13,7 +14,6 @@
         class="portfolio__project"
       ></project-widget>
     </div>
-
   </div>
 </template>
 
@@ -21,6 +21,7 @@
   import ProjectWidget from '@/components/portfolio/ProjectWidget';
   import AddProject from '@/components/portfolio/AddProject';
 
+  import ME_QUERY from '@/graphql/gql/meQuery.gql';
   import PROJECTS_QUERY from '@/graphql/gql/allProjects.gql';
   import DELETE_PROJECT_MUTATION from '@/graphql/gql/mutations/deleteProject.gql';
   import UPDATE_PROJECT_MUTATION from '@/graphql/gql/mutations/updateProject.gql';
@@ -38,13 +39,24 @@
           return this.$getRidOfEdges(data).projects
         }
       },
-    },
-
-    data() {
-      return {
-        projects: []
+      me: {
+        query: ME_QUERY
       }
     },
+
+    data () {
+      return {
+        projects: [],
+        me: {}
+      }
+    },
+
+    computed: {
+      canManageContent () {
+        return this.me.permissions.includes('users.can_manage_school_class_content');
+      }
+    },
+
     methods: {
       deleteProject(id) {
         this.$apollo.mutate({

From f9f1b6819c70945e30b448ad877d0981b9202b87 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 15:37:59 +0100
Subject: [PATCH 5/9] use id as condition for menu, add owner name

---
 client/src/components/portfolio/ProjectWidget.vue | 14 ++++++++++----
 client/src/graphql/gql/fragments/projectParts.gql |  5 +++++
 client/src/pages/portfolio.vue                    |  6 +++---
 3 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index 2ac339ec..00403945 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -4,9 +4,9 @@
             <h3 class="project-widget__title">{{title}}</h3>
 
       <entry-count-widget entry-count="4"></entry-count-widget>
-      <owner-widget name="Hans Muster"></owner-widget>
+      <owner-widget :name="owner"></owner-widget>
     </router-link>
-    <widget-footer v-if="!isTeacher">
+    <widget-footer v-if="isOwner">
       <template slot-scope="scope">
           <li class="popover-links__link"><a @click="$emit('delete', id)">Projekt löschen</a></li>
           <li class="popover-links__link"><a @click="$emit('edit', id)">Projekt bearbeiten</a></li>
@@ -23,7 +23,7 @@
   import WidgetFooter from '@/components/WidgetFooter';
 
   export default {
-    props: ['title', 'appearance', 'slug', 'id', 'final', 'isTeacher'],
+    props: ['title', 'appearance', 'slug', 'id', 'final', 'student', 'userId'],
 
     components: {
       WidgetFooter,
@@ -32,8 +32,14 @@
     },
 
     computed: {
-      widgetClass() {
+      widgetClass () {
         return `project-widget--${this.appearance}`;
+      },
+      isOwner () {
+        return this.student.id === this.userId;
+      },
+      owner () {
+        return `${this.student.firstName} ${this.student.lastName}`
       }
     },
 
diff --git a/client/src/graphql/gql/fragments/projectParts.gql b/client/src/graphql/gql/fragments/projectParts.gql
index c0d1a8b5..9d0bafe3 100644
--- a/client/src/graphql/gql/fragments/projectParts.gql
+++ b/client/src/graphql/gql/fragments/projectParts.gql
@@ -6,4 +6,9 @@ fragment ProjectParts on ProjectNode {
   slug
   objectives
   final
+  student {
+    firstName
+    lastName
+    id
+  }
 }
diff --git a/client/src/pages/portfolio.vue b/client/src/pages/portfolio.vue
index 2197a961..d4bf8304 100644
--- a/client/src/pages/portfolio.vue
+++ b/client/src/pages/portfolio.vue
@@ -6,7 +6,7 @@
       <project-widget
         v-for="project in projects"
         v-bind="project"
-        :isTeacher="canManageContent"
+        :userId="userId"
         @delete="deleteProject"
         @updateShare="updateShareState"
         @edit="editProject"
@@ -52,8 +52,8 @@
     },
 
     computed: {
-      canManageContent () {
-        return this.me.permissions.includes('users.can_manage_school_class_content');
+      userId () {
+        return this.me.id;
       }
     },
 

From d03524e803559c2e0d4b76e254322bf08e3f0550 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 15:56:27 +0100
Subject: [PATCH 6/9] add entry count to project widget

---
 client/src/components/portfolio/ProjectWidget.vue | 4 ++--
 client/src/graphql/gql/fragments/projectParts.gql | 1 +
 server/portfolio/schema.py                        | 4 ++++
 server/rooms/schema.py                            | 1 +
 4 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/client/src/components/portfolio/ProjectWidget.vue b/client/src/components/portfolio/ProjectWidget.vue
index 00403945..4c374265 100644
--- a/client/src/components/portfolio/ProjectWidget.vue
+++ b/client/src/components/portfolio/ProjectWidget.vue
@@ -3,7 +3,7 @@
     <router-link :to="{name: 'project', params: {slug: slug}}" tag="div" class="project-widget__content">
             <h3 class="project-widget__title">{{title}}</h3>
 
-      <entry-count-widget entry-count="4"></entry-count-widget>
+      <entry-count-widget :entry-count="entriesCount"></entry-count-widget>
       <owner-widget :name="owner"></owner-widget>
     </router-link>
     <widget-footer v-if="isOwner">
@@ -23,7 +23,7 @@
   import WidgetFooter from '@/components/WidgetFooter';
 
   export default {
-    props: ['title', 'appearance', 'slug', 'id', 'final', 'student', 'userId'],
+    props: ['title', 'appearance', 'slug', 'id', 'final', 'student', 'entriesCount', 'userId'],
 
     components: {
       WidgetFooter,
diff --git a/client/src/graphql/gql/fragments/projectParts.gql b/client/src/graphql/gql/fragments/projectParts.gql
index 9d0bafe3..4f7d27d5 100644
--- a/client/src/graphql/gql/fragments/projectParts.gql
+++ b/client/src/graphql/gql/fragments/projectParts.gql
@@ -11,4 +11,5 @@ fragment ProjectParts on ProjectNode {
     lastName
     id
   }
+  entriesCount
 }
diff --git a/server/portfolio/schema.py b/server/portfolio/schema.py
index 9e7b9864..780ff14b 100644
--- a/server/portfolio/schema.py
+++ b/server/portfolio/schema.py
@@ -10,6 +10,7 @@ from users.models import UserRole, Role
 
 class ProjectNode(DjangoObjectType):
     pk = graphene.Int()
+    entries_count = graphene.Int()
 
     class Meta:
         model = Project
@@ -19,6 +20,9 @@ class ProjectNode(DjangoObjectType):
     def resolve_pk(self, *args, **kwargs):
         return self.id
 
+    def resolve_entries_count(self, *args, **kwargs):
+        return self.entries.count()
+
 
 class ProjectEntryNode(DjangoObjectType):
     class Meta:
diff --git a/server/rooms/schema.py b/server/rooms/schema.py
index e224a5ec..476a6e8f 100644
--- a/server/rooms/schema.py
+++ b/server/rooms/schema.py
@@ -11,6 +11,7 @@ from users.schema import UserNode
 
 logger = logging.getLogger(__name__)
 
+
 class RoomEntryNode(DjangoObjectType):
     pk = graphene.Int()
     author = UserNode()

From ab6b1d76beada486ce6c9f427a430288974d7906 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Wed, 27 Mar 2019 16:02:28 +0100
Subject: [PATCH 7/9] add missing settings to test configuration

---
 server/core/settings_test.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/server/core/settings_test.py b/server/core/settings_test.py
index 1ad6e870..a480a60b 100644
--- a/server/core/settings_test.py
+++ b/server/core/settings_test.py
@@ -10,3 +10,7 @@ class DisableMigrations(object):
 
 
 MIGRATION_MODULES = DisableMigrations()
+
+# Email Settings
+SENDGRID_API_KEY = ""
+EMAIL_BACKEND = 'django.core.mail.backends.dummy.EmailBackend'

From ffc63328940abaccb896a35425656b5f094f1f10 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Thu, 28 Mar 2019 08:42:37 +0100
Subject: [PATCH 8/9] add edit project page

---
 .../src/components/portfolio/EditProject.vue  | 41 +++++++++++++++++++
 client/src/pages/editProject.vue              | 37 +++++++++++++++++
 client/src/router/index.js                    |  2 +
 3 files changed, 80 insertions(+)
 create mode 100644 client/src/components/portfolio/EditProject.vue
 create mode 100644 client/src/pages/editProject.vue

diff --git a/client/src/components/portfolio/EditProject.vue b/client/src/components/portfolio/EditProject.vue
new file mode 100644
index 00000000..343c7a53
--- /dev/null
+++ b/client/src/components/portfolio/EditProject.vue
@@ -0,0 +1,41 @@
+<template>
+  <project-form
+    :project="project"
+    @save="updateProject"
+  ></project-form>
+</template>
+
+<script>
+  import ProjectForm from '@/components/portfolio/ProjectForm';
+
+  import UPDATE_PROJECT_MUTATION from '@/graphql/gql/mutations/updateProject.gql';
+
+  export default {
+    props: ['project'],
+
+    components: {
+      ProjectForm
+    },
+
+    methods: {
+      updateProject(project) {
+        this.$apollo.mutate({
+          mutation: UPDATE_PROJECT_MUTATION,
+          variables: {
+            input: {
+              project: {
+                id: project.id,
+                title: project.title,
+                description: project.description,
+                appearance: project.appearance,
+                objectives: project.objectives
+              }
+            }
+          }
+        }).then(() => {
+          this.$router.push('/portfolio');
+        });
+      }
+    },
+  }
+</script>
diff --git a/client/src/pages/editProject.vue b/client/src/pages/editProject.vue
new file mode 100644
index 00000000..3eeb50fc
--- /dev/null
+++ b/client/src/pages/editProject.vue
@@ -0,0 +1,37 @@
+<template>
+  <div class="edit-project-page">
+    <edit-project :project="project" v-if="this.project.id"></edit-project>
+  </div>
+</template>
+
+<script>
+  // todo: refactor this, we don't need 2 components, remove editRoom or EditRoom component
+  import EditProject from '@/components/portfolio/EditProject';
+
+  import PROJECT_QUERY from '@/graphql/gql/projectQuery.gql';
+
+  export default {
+    props: ['id'],
+
+    components: {
+      EditProject
+    },
+
+    data() {
+      return {
+        project: {}
+      }
+    },
+
+    apollo: {
+      project: {
+        query: PROJECT_QUERY,
+        variables() {
+          return {
+            id: this.id
+          }
+        }
+      }
+    }
+  }
+</script>
diff --git a/client/src/router/index.js b/client/src/router/index.js
index a50c9f60..4fd97bd1 100644
--- a/client/src/router/index.js
+++ b/client/src/router/index.js
@@ -18,6 +18,7 @@ import start from '@/pages/start'
 import submission from '@/pages/studentSubmission'
 import portfolio from '@/pages/portfolio'
 import project from '@/pages/project'
+import editProject from '@/pages/editProject'
 import newProject from '@/pages/newProject'
 
 const routes = [
@@ -53,6 +54,7 @@ const routes = [
   {path: '/portfolio', name: 'portfolio', component: portfolio},
   {path: '/portfolio/:slug', name: 'project', component: project, props: true},
   {path: '/new-project/', name: 'new-project', component: newProject},
+  {path: '/edit-project/:id', name: 'edit-project', component: editProject, props: true},
   {
     path: '/book',
     name: 'book',

From 0ee5f65b107ea440acce74003baa3cf240184bf1 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Tue, 9 Apr 2019 08:27:54 +0200
Subject: [PATCH 9/9] add actions to room widget

---
 client/src/components/rooms/RoomEntry.vue | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/client/src/components/rooms/RoomEntry.vue b/client/src/components/rooms/RoomEntry.vue
index a7e3ebe1..f591459e 100644
--- a/client/src/components/rooms/RoomEntry.vue
+++ b/client/src/components/rooms/RoomEntry.vue
@@ -4,13 +4,13 @@
       <a @click="showMenu = !showMenu" class="room-entry__more-link">
         <ellipses class="room-entry__ellipses"></ellipses>
       </a>
-      <widget-popover entity="Eintrag"
-                      @delete="deleteRoomEntry"
-                      @edit="editRoomEntry"
-                      @hide-me="showMenu = false"
+      <widget-popover @hide-me="showMenu = false"
                       :id="id"
                       class="room-entry__popover"
-                      v-if="showMenu"></widget-popover>
+                      v-if="showMenu">
+        <li class="popover-links__link"><a @click="deleteRoomEntry(id)">Raum löschen</a></li>
+        <li class="popover-links__link"><a @click="editRoomEntry(id)">Raum bearbeiten</a></li>
+       </widget-popover>
     </div>
     <router-link :to="{name: 'article', params: { slug: slug }}" tag="div" class="room-entry__router-link">
       <div class="room-entry__header" v-if="image">