wip: workgroup description

client/src/components/AppLoader.tsx

@@ -2,6 +2,8 @@ import React, { FunctionComponent } from "react";
 
 export const AppLoader:FunctionComponent = () => {
   return (
-    <div className="loader"></div>
+    <div className="app-loader">
+      <i className="fas fa-spinner fa-spin fa-5x"></i>
+    </div>
   )
 }

client/src/components/ItemPanel.tsx

@@ -1,6 +1,5 @@
 import React, { FunctionComponent, useState, useEffect } from "react";
 import { Link } from "react-router-dom";
-import { WithLoader } from "./WithLoader";
 
 export interface Item {
   id: string

client/src/components/ProfilePage/ProfilePage.tsx

@@ -2,19 +2,16 @@ import React from 'react';
 import { Page } from '../Page';
 import { UserForm } from '../UserForm';
 import { User } from '../../types/user';
-import { useUserProfileQuery } from '../../gql/queries/profile';
+import { useUserProfile } from '../../gql/queries/profile';
 import { useUpdateUserProfileMutation } from '../../gql/mutations/profile';
-import { WithLoader } from '../WithLoader';
 
 export function ProfilePage() {
-  const userProfileQuery = useUserProfileQuery();
+  const { user, loading } = useUserProfile();
   const [ updateProfile, updateUserProfileMutation ] = useUpdateUserProfileMutation();
-  const isLoading = updateUserProfileMutation.loading || userProfileQuery.loading;
-
-  const { userProfile } = (userProfileQuery.data || {});
+  const isLoading = updateUserProfileMutation.loading || loading;
 
   const onUserChange = (user: User) => {
-    if (userProfile.name !== user.name) {
+    if (user.name !== user.name) {
       updateProfile({ variables: {changes: { name: user.name }}});
     }
   };
@@ -27,11 +24,7 @@ export function ProfilePage() {
             <div className="column is-6 is-offset-3">
               <div className="box">
                 <h2 className="is-size-2 subtitle">Mon profil</h2>
-                <WithLoader loading={isLoading || !userProfile}>
-                {
-                  <UserForm onChange={onUserChange} user={userProfile} /> 
-                }
-                </WithLoader>
+                { !isLoading ? <UserForm onChange={onUserChange} user={user} /> : null }
               </div>
             </div>
           </div>

client/src/components/WithLoader.tsx

@@ -1,18 +0,0 @@
-import React, { Fragment, PropsWithChildren, FunctionComponent } from 'react';
-
-export interface WithLoaderProps {
-  loading?: boolean|boolean[]
-}
-
-export const WithLoader: FunctionComponent<WithLoaderProps> = ({ loading, children }) => {
-  const isLoading = Array.isArray(loading) ? loading.some(l => l) : loading;
-  return (
-    <Fragment>
-      {
-        isLoading ?
-        <div>Chargement</div> :
-        children
-      }
-    </Fragment>
-  )
-}

client/src/components/WorkgroupPage/InfoForm.tsx

@@ -14,6 +14,7 @@ export function InfoForm({ workgroup, onChange }: InfoFormProps) {
     workgroup: {
       id: workgroup && workgroup.id ? workgroup.id : '',
       name: workgroup && workgroup.name ? workgroup.name : '',
+      description: workgroup && workgroup.description ? workgroup.description : '',
       createdAt: workgroup && workgroup.createdAt ? workgroup.createdAt : null,
       closedAt: workgroup && workgroup.closedAt ? workgroup.closedAt : null,
     }
@@ -34,6 +35,7 @@ export function InfoForm({ workgroup, onChange }: InfoFormProps) {
       workgroup: {
         id: workgroup && workgroup.id ? workgroup.id : '',
         name: workgroup && workgroup.name ? workgroup.name : '',
+        description: workgroup && workgroup.description ? workgroup.description : '',
         createdAt: workgroup && workgroup.createdAt ? workgroup.createdAt : null,
         closedAt: workgroup && workgroup.closedAt ? workgroup.closedAt : null,
       }
@@ -76,6 +78,14 @@ export function InfoForm({ workgroup, onChange }: InfoFormProps) {
             onChange={onWorkgroupAttrChange.bind(null, "name")} />
         </div>
       </div>
+      <div className="field">
+        <label className="label">Nom du groupe</label>
+        <div className="control">
+          <textarea className="input" value={state.workgroup.description}
+            disabled={!isAuthorized}
+            onChange={onWorkgroupAttrChange.bind(null, "description")}></textarea>
+        </div>
+      </div>
       {
         state.workgroup.createdAt ?
         <div className="field">

client/src/components/WorkgroupPage/InfoPanel.tsx

@@ -1,8 +1,6 @@
 import React, { FunctionComponent } from 'react';
-import { User } from '../../types/user';
 import { Workgroup } from '../../types/workgroup';
 import { InfoForm } from './InfoForm';
-import { WithLoader } from '../WithLoader';
 import { useUpdateWorkgroupMutation, useCreateWorkgroupMutation } from '../../gql/mutations/workgroups';
 import { useHistory } from 'react-router';
 
@@ -43,9 +41,7 @@ export const InfoPanel: FunctionComponent<InfoPanelProps> = ({ workgroup }) => {
         Informations
       </p>
       <div className="panel-block">
-        <WithLoader loading={isLoading}>
-          <InfoForm workgroup={workgroup} onChange={onWorkgroupChange} />
-        </WithLoader>
+        { !isLoading ? <InfoForm workgroup={workgroup} onChange={onWorkgroupChange} /> : null }
       </div>
     </nav>
   );

client/src/gql/mutations/workgroups.tsx

@@ -7,6 +7,7 @@ mutation updateWorkgroup($workgroupId: ID!, $changes: WorkgroupChanges!) {
   updateWorkgroup(workgroupId: $workgroupId, changes: $changes) {
     id,
     name,
+    description,
     createdAt,
     closedAt,
     members {
@@ -26,6 +27,7 @@ mutation createWorkgroup($changes: WorkgroupChanges!) {
   createWorkgroup(changes: $changes) {
     id,
     name,
+    description,
     createdAt,
     closedAt,
     members {
@@ -47,6 +49,7 @@ mutation joinWorkgroup($workgroupId: ID!) {
   joinWorkgroup(workgroupId: $workgroupId) {
     id,
     name,
+    description,
     createdAt,
     closedAt,
     members {
@@ -78,6 +81,7 @@ mutation leaveWorkgroup($workgroupId: ID!) {
   leaveWorkgroup(workgroupId: $workgroupId) {
     id,
     name,
+    description,
     createdAt,
     closedAt,
     members {
@@ -117,6 +121,7 @@ mutation closeWorkgroup($workgroupId: ID!) {
   closeWorkgroup(workgroupId: $workgroupId) {
     id,
     name,
+    description,
     createdAt,
     closedAt,
     members {

client/src/gql/queries/workgroups.tsx

@@ -7,6 +7,7 @@ export const QUERY_WORKGROUP = gql`
     workgroups(filter: $filter) {
       id,
       name,
+      description,
       createdAt,
       closedAt,
       members {

client/src/sass/_loader.scss

@@ -1,44 +1,16 @@
-.loader-container {
-    display: flex;
-    width: 100%;
-    justify-content: center;
-    height: 100%;
-    align-items: center;
-}
+.app-loader {
+  @extend body;
 
-.lds-ripple {
-    display: inline-block;
-    position: relative;
-    width: 80px;
-    height: 80px;
-    transform: scale(2);
-}
-
-.lds-ripple div {
-    position: absolute;
-    border: 4px solid $grey;
-    opacity: 1;
-    border-radius: 50%;
-    animation: lds-ripple 1s cubic-bezier(0, 0.2, 0.8, 1) infinite;
-}
-
-.lds-ripple div:nth-child(2) {
-    animation-delay: -0.5s;
-}
-
-@keyframes lds-ripple {
-    0% {
-      top: 36px;
-      left: 36px;
-      width: 0;
-      height: 0;
-      opacity: 1;
-    }
-    100% {
-      top: 0px;
-      left: 0px;
-      width: 72px;
-      height: 72px;
-      opacity: 0;
-    }
-  }
+  display: flex;
+  position: absolute;
+  z-index: 10000;
+  top: 0;
+  bottom: 0;
+  left: 0;
+  right: 0;
+  height: 100vh;
+  width: 100vw;
+  align-items: center;
+  justify-content: center;
+  flex-direction: column;
+}

client/src/types/workgroup.ts

@@ -2,6 +2,7 @@ import { User } from "./user";
 export interface Workgroup {
   id: string
   name: string
+  description: string
   createdAt: Date
   closedAt: Date
   members: User[]

cmd/server/container.go

@@ -79,8 +79,9 @@ func getServiceContainer(ctx context.Context, conf *config.Config) (*service.Con
 	// Define default cookie options
 	sessionStore.SessionOpts.Path = "/"
 	sessionStore.SessionOpts.HttpOnly = true
+	sessionStore.SessionOpts.Secure = conf.HTTP.CookieSecure
 	sessionStore.SessionOpts.MaxAge = conf.HTTP.CookieMaxAge
-	sessionStore.SessionOpts.SameSite = http.SameSiteStrictMode
+	sessionStore.SessionOpts.SameSite = http.SameSiteLaxMode
 
 	ctn.Provide(
 		session.ServiceName,

cmd/server/migration.go

@@ -27,7 +27,8 @@ func applyMigration(ctx context.Context, ctn *service.Container) error {
 
 	// Register available migrations
 	migr.Register(
-		m000initialSchema(),
+		m000InitialSchema(),
+		m001WorkgroupDescription(),
 	)
 
 	currentVersion, err := migr.CurrentVersion(ctx)
@@ -84,7 +85,7 @@ var initialModels = []interface{}{
 	&model.Event{},
 }
 
-func m000initialSchema() orm.Migration {
+func m000InitialSchema() orm.Migration {
 	return orm.NewDBMigration(
 		"00_initial_schema",
 		func(ctx context.Context, tx *gorm.DB) error {
@@ -107,3 +108,23 @@ func m000initialSchema() orm.Migration {
 		},
 	)
 }
+
+func m001WorkgroupDescription() orm.Migration {
+	return orm.NewDBMigration(
+		"01_workgroup_description",
+		func(ctx context.Context, tx *gorm.DB) error {
+			if err := tx.AutoMigrate(&model.Workgroup{}).Error; err != nil {
+				return errors.WithStack(err)
+			}
+
+			return nil
+		},
+		func(ctx context.Context, tx *gorm.DB) error {
+			if err := tx.Model(&model.Workgroup{}).DropColumn("description").Error; err != nil {
+				return errors.WithStack(err)
+			}
+
+			return nil
+		},
+	)
+}

debian/compat

@@ -1 +0,0 @@
-9

debian/control

@@ -1,14 +0,0 @@
-Source: daddy
-Section: unknown
-Priority: optional
-Maintainer: Cadoles <contact@cadoles.com>
-Build-Depends: debhelper (>= 8.0.0), wget, ca-certificates, tar
-Standards-Version: 3.9.4
-Homepage: http://forge.cadoles.com/Cadoles/daddy
-Vcs-Git: http://forge.cadoles.com/Cadoles/daddy.git
-Vcs-Browser: http://forge.cadoles.com/Cadoles/daddy
-
-Package: daddy
-Architecture: amd64
-Depends: ${shlibs:Depends}, ${misc:Depends}
-Description: Daddy app

debian/daddy.links

@@ -1 +0,0 @@
-/etc/daddy/client-config.js /usr/share/daddy/public/config.js

debian/daddy.service

@@ -1,11 +0,0 @@
-[Unit]
-Description=Daddy app
-After=network-online.target
-
-[Service]
-Type=simple
-ExecStart=/usr/bin/daddy -workdir /usr/share/daddy -config /etc/daddy/config.yml
-Restart=on-failure
-
-[Install]
-WantedBy=multi-user.target

debian/rules

@@ -1,56 +0,0 @@
-#!/usr/bin/make -f
-# -*- makefile -*-
-
-# Uncomment this to turn on verbose mode.
-export DH_VERBOSE=1
-
-GO_VERSION := 1.13.5
-OS := linux
-ARCH := amd64
-GOPATH=$(HOME)/go
-
-ifeq (, $(shell which go 2>/dev/null))
-override_dh_auto_build: install-go
-
-override_dh_auto_clean: install-go
-endif
-
-ifeq (, $(shell which node 2>/dev/null))
-override_dh_auto_build: install-nodejs
-endif
-
-%:
-	dh $@ --with systemd
-
-override_dh_auto_build: $(GOPATH)
-	GOPATH=$(GOPATH) PATH="$(PATH):/usr/local/go/bin:$(GOPATH)/bin" make deps
-	GOPATH=$(GOPATH) PATH="$(PATH):/usr/local/go/bin:$(GOPATH)/bin" ARCH_TARGETS=$(ARCH) make release
-
-$(GOPATH):
-	mkdir -p $(GOPATH)
-	
-install-go:
-	wget -nc https://dl.google.com/go/go$(GO_VERSION).$(OS)-$(ARCH).tar.gz
-	tar -C /usr/local -xzf go$(GO_VERSION).$(OS)-$(ARCH).tar.gz
-
-install-nodejs:
-	wget -O- https://deb.nodesource.com/setup_12.x | bash -
-	apt-get install -y nodejs
-
-override_dh_auto_install:
-
-	mkdir -p debian/daddy/usr/share/daddy
-	mkdir -p debian/daddy/etc/daddy
-	mkdir -p debian/daddy/usr/bin
-
-	cp -r release/server-$(OS)-$(ARCH)/* debian/daddy/usr/share/daddy/
-	
-	mv debian/daddy/usr/share/daddy/bin/server debian/daddy/usr/bin/daddy
-	mv debian/daddy/usr/share/daddy/server.conf debian/daddy/etc/daddy/config.yml
-	mv debian/daddy/usr/share/daddy/public/config.js debian/daddy/etc/daddy/client-config.js
-
-	install -d debian/daddy
-
-override_dh_strip:
-
-override_dh_auto_test:

debian/source/format

@@ -1 +0,0 @@
-3.0 (native)

go.mod

@@ -3,7 +3,7 @@ module forge.cadoles.com/Cadoles/daddy
 go 1.14
 
 require (
-	forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013085949-5d5592098f13
+	forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013125038-8d8d1519a52d
 	forge.cadoles.com/wpetit/hydra-passwordless v0.0.0-20200908094025-38ac4422dddc // indirect
 	github.com/99designs/gqlgen v0.11.3
 	github.com/alecthomas/chroma v0.8.1 // indirect

go.sum

@@ -39,6 +39,10 @@ forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20200619080035-4bbf7b016032 h1:qTYaLP
 forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20200619080035-4bbf7b016032/go.mod h1:gkfqGyk7fCj2Z0ngEOCJ3K0FVmqft/8dFV/OnYT1vec=
 forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013085949-5d5592098f13 h1:gZCo9pX3I3A0xVvXkbhdTDBbW44CypCxjiP5LtNV5bo=
 forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013085949-5d5592098f13/go.mod h1:phGAWHUGKNZj044478BvRg0jk049uK1IiX2Amh8krAk=
+forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013111944-d43b43b636ed h1:7dTCXOGxvAulu9vnOjpt2cTgsuxMHX4FH795/JJgo08=
+forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013111944-d43b43b636ed/go.mod h1:phGAWHUGKNZj044478BvRg0jk049uK1IiX2Amh8krAk=
+forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013125038-8d8d1519a52d h1:o+Ppy/MyT5UgbtUYI2J1YqS3iuThxOuNFenYoPgKZKk=
+forge.cadoles.com/wpetit/goweb-oidc v0.0.0-20201013125038-8d8d1519a52d/go.mod h1:phGAWHUGKNZj044478BvRg0jk049uK1IiX2Amh8krAk=
 forge.cadoles.com/wpetit/hydra-passwordless v0.0.0-20200908094025-38ac4422dddc h1:9gc/1qizPtK6/iMVlizknWUFii75ntl2xSUV/FSC92Y=
 forge.cadoles.com/wpetit/hydra-passwordless v0.0.0-20200908094025-38ac4422dddc/go.mod h1:nANHORi270d5jDXjeJ7B3pMgK9R4J0/17p1IIc+rhOk=
 github.com/99designs/gqlgen v0.11.3 h1:oFSxl1DFS9X///uHV3y6CEfpcXWrDUxVblR4Xib2bs4=

internal/config/config.go

@@ -47,6 +47,7 @@ type HTTPConfig struct {
 	CookieAuthenticationKey string     `yaml:"cookieAuthenticationKey" env:"HTTP_COOKIE_AUTHENTICATION_KEY"`
 	CookieEncryptionKey     string     `yaml:"cookieEncryptionKey" env:"HTTP_COOKIE_ENCRYPTION_KEY"`
 	CookieMaxAge            int        `yaml:"cookieMaxAge" env:"HTTP_COOKIE_MAX_AGE"`
+	CookieSecure            bool       `yaml:"cookieSecure" env:"HTTP_COOKIE_SECURE"`
 	TemplateDir             string     `yaml:"templateDir" env:"HTTP_TEMPLATE_DIR"`
 	PublicDir               string     `yaml:"publicDir" env:"HTTP_PUBLIC_DIR"`
 	FrontendURL             string     `yaml:"frontendURL" env:"HTTP_FRONTEND_URL"`

internal/model/workgroup.go

@@ -10,9 +10,10 @@ const ObjectTypeWorkgroup = "workgroup"
 
 type Workgroup struct {
 	gorm.Model
-	Name     *string   `json:"name"`
-	ClosedAt time.Time `json:"closedAt"`
-	Members  []*User   `gorm:"many2many:users_workgroups;association_autoupdate:false"`
+	Name      *string   `json:"name"`
+	Workgroup *string   `json:"workgroup"`
+	ClosedAt  time.Time `json:"closedAt"`
+	Members   []*User   `gorm:"many2many:users_workgroups;association_autoupdate:false"`
 }
 
 func (w *Workgroup) ObjectID() uint {

internal/route/login.go

@@ -68,9 +68,8 @@ func handleLoginCallback(w http.ResponseWriter, r *http.Request) {
 	db := orm.Must(ctn).DB()
 	repo := model.NewUserRepository(db)
 
-	user, err := repo.CreateOrConnectUser(ctx, claims.Email)
-	if err != nil {
-		panic(errors.Wrap(err, "could not upsert user"))
+	user := &model.User{
+		Email: claims.Email,
 	}
 
 	authorized, err := auth.Authorize(user)
@@ -85,6 +84,11 @@ func handleLoginCallback(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
+	user, err = repo.CreateOrConnectUser(ctx, claims.Email)
+	if err != nil {
+		panic(errors.Wrap(err, "could not upsert user"))
+	}
+
 	if err := session.SaveUserEmail(w, r, claims.Email); err != nil {
 		panic(errors.WithStack(err))
 	}

misc/debian/nginx/daddy.conf

@@ -1,15 +0,0 @@
-server {
-    listen 80 default_server;
-    server_name daddy.local;
-    
-    root /usr/share/daddy/client/public;
-    index index.html;
-    
-    location / {
-        try_files $uri /index.html =404;
-    }
-
-    location /api/v1/graphql {
-        proxy_pass http://127.0.0.1:8080/api/v1/graphql;
-    }
-}

misc/debian/systemd/daddy.env

@@ -1 +0,0 @@
-GO_ENV=prod