DGNum/gestioCOF
15
2
Fork 1
Code Issues 78 Pull requests 14 Projects Releases 12 Packages Wiki Activity

La K-Fêt est-elle ouverte ? #543

Merged
delobell merged 0 commits from aureplop/kfet_open into master 2017-06-23 00:29:08 +02:00
Conversation 25 Commits - Files changed -
delobell commented 2017-06-21 07:32:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Fin du job entrepris par @narmanli (!184)

  • Un témoin indiquant si la K-Fêt est ouverte ou fermée est ajoutée à la barre de navigation de kfet.
  • Les mises à jour sont transmises en temps réel par websockets.
  • Un administrateur peut forcer la fermeture. Les informations brutes restent néanmoins visibles pour les membres de l'équipe.
Fin du job entrepris par @narmanli (!184) - Un témoin indiquant si la K-Fêt est ouverte ou fermée est ajoutée à la barre de navigation de kfet. - Les mises à jour sont transmises en temps réel par websockets. - Un administrateur peut forcer la fermeture. Les informations brutes restent néanmoins visibles pour les membres de l'équipe.
delobell commented 2017-06-21 07:35:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Sans ça les tests échouent quand un WSClient est utilisé. Le client crée un object FakeRequest sans attribut GET...

Sans ça les tests échouent quand un `WSClient` est utilisé. Le client crée un object `FakeRequest` sans attribut `GET`...
mpepin commented 2017-06-21 20:00:01 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Typo: manuellement

Typo: manuellement
mpepin commented 2017-06-21 20:01:25 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

typo: different

typo: different
mpepin commented 2017-06-21 20:04:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi gérer le cas user is None ? Peut-il seulement arriver ?

Pourquoi gérer le cas `user is None` ? Peut-il seulement arriver ?
mpepin commented 2017-06-21 20:09:53 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ok j'imagine que ce comportement est lié à mon commentaire précédent. Du coup ma question est : pourquoi veut on ça ?

Ok j'imagine que ce comportement est lié à mon commentaire précédent. Du coup ma question est : pourquoi veut on ça ?
mpepin commented 2017-06-21 20:11:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je ne suis pas ultra fan des noms de variables à une lettre… On dirait du code OCaml :P

Je ne suis pas ultra fan des noms de variables à une lettre… On dirait du code OCaml :P
mpepin commented 2017-06-21 20:14:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ahahah ;)

Ahahah ;)
mpepin commented 2017-06-21 20:16:02 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Fair enough

Fair enough
mpepin commented 2017-06-21 20:22:14 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

À quelques détails cosmétiques je n'ai rien à dire. Chapeau pour les tests !

À quelques détails cosmétiques je n'ai rien à dire. Chapeau pour les tests !
delobell commented 2017-06-21 21:11:26 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je crois pas que le cas arrive en l'état.
Mais ça me semblait intéressant de pouvoir juste appeler export() pour du débugage ou du logging par exemple.

Je crois pas que le cas arrive en l'état. Mais ça me semblait intéressant de pouvoir juste appeler `export()` pour du débugage ou du logging par exemple.
delobell commented 2017-06-21 21:12:37 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pour les tests, je me lâche un peu.

Pour les tests, je me lâche un peu.
mpepin commented 2017-06-21 21:24:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Fair enough tant qu'on n'oublie pas de passer un User là où ça sert

Fair enough tant qu'on n'oublie pas de passer un User là où ça sert
mpepin commented 2017-06-21 21:29:33 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

fair enough

fair enough
delobell commented 2017-06-21 23:31:37 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
delobell commented 2017-06-21 23:31:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 782e1056 - typo [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/239/diffs?diff_id=529&start_sha=b8110c11a40454bd26479ebf2065159d1286ce35)
mpepin commented 2017-06-22 03:11:19 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Finalement j'ai un petit problème avec ça : est-ce qu'on ne voudrait pas mettre une auth là dessus ? Je verrais bien un truc simple avec juste un clef secrète passée en paramètre get et importée avec les autres secrets dans les settings (bien sûr il faudra s'assure que le pi fait ses requêtes en HTTPS)

Finalement j'ai un petit problème avec ça : est-ce qu'on ne voudrait pas mettre une auth là dessus ? Je verrais bien un truc simple avec juste un clef secrète passée en paramètre get et importée avec les autres secrets dans les settings (bien sûr il faudra s'assure que le pi fait ses requêtes en HTTPS)
mpepin commented 2017-06-22 03:15:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

En demi écran je n'ai pas la popup pour fermer manuellement. C'est pas critique mais si c'est facile à gérer ça serait pas mal.

Edit: ça vaut pour l'affichage mobile aussi

En demi écran je n'ai pas la popup pour fermer manuellement. C'est pas critique mais si c'est facile à gérer ça serait pas mal. **Edit**: ça vaut pour l'affichage mobile aussi
delobell commented 2017-06-22 03:48:13 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça me chagrine aussi. D'après le dépôt du raspi, un token est passé en paramètre GET et https est déjà utilisé.
On peut mettre ce token en secret, mais faudrait aussi mieux le gérer sur le raspi, en le settant dans l'environnement. Pour l'instant il est en dur dans le code.

Ça me chagrine aussi. D'après le dépôt du raspi, un token est passé en paramètre GET et https est déjà utilisé. On peut mettre ce token en secret, mais faudrait aussi mieux le gérer sur le raspi, en le settant dans l'environnement. Pour l'instant il est en dur dans le code.
mpepin commented 2017-06-22 03:53:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

J'ai envie de dire que le raspi est un problème secondaire. Si GestioCOF sait gérer ça c'est bien, après setup le raspi correctement (et sans gitter la clef…) c'est pas très compliqué

J'ai envie de dire que le raspi est un problème secondaire. Si GestioCOF sait gérer ça c'est bien, après setup le raspi correctement (et sans gitter la clef…) c'est pas très compliqué
delobell commented 2017-06-22 16:40:47 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 3 commits

  • 98f5f0c3 - update refresh/unknown interval
  • 19847ac9 - add token check to raw_open edit view
  • 5673fabe - Better status management.

Compare with previous version

added 3 commits * 98f5f0c3 - update refresh/unknown interval * 19847ac9 - add token check to raw_open edit view * 5673fabe - Better status management. [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/239/diffs?diff_id=530&start_sha=782e105644f83ca906e59d68bce23d47d9c0a241)
delobell commented 2017-06-22 17:00:04 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 815ba506 - fix for small devices [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/239/diffs?diff_id=531&start_sha=5673fabeff1ba72755ee656cbc397ea202157973)
mpepin commented 2017-06-23 00:28:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
mpepin commented 2017-06-23 00:29:09 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
mpepin commented 2017-06-23 00:29:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit b49d96d18f

mentioned in commit b49d96d18fa561a1d4dc2a3361b3d48d35ab0972
mpepin commented 2017-06-25 17:04:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #65

mentioned in issue #65
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#543
No description provided.
Delete branch "aureplop/kfet_open"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?