La K-Fêt est-elle ouverte ? #543
No reviewers
Labels
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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#543
Loading…
Reference in a new issue
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?
Fin du job entrepris par @narmanli (!184)
Sans ça les tests échouent quand un
WSClient
est utilisé. Le client crée un objectFakeRequest
sans attributGET
...Typo: manuellement
typo: different
Pourquoi gérer le cas
user is None
? Peut-il seulement arriver ?Ok j'imagine que ce comportement est lié à mon commentaire précédent. Du coup ma question est : pourquoi veut on ça ?
Je ne suis pas ultra fan des noms de variables à une lettre… On dirait du code OCaml :P
Ahahah ;)
Fair enough
À quelques détails cosmétiques je n'ai rien à dire. Chapeau pour les tests !
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.Pour les tests, je me lâche un peu.
Fair enough tant qu'on n'oublie pas de passer un User là où ça sert
fair enough
resolved all discussions
added 1 commit
782e1056
- typoCompare with previous version
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)
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
Ç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.
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é
added 3 commits
98f5f0c3
- update refresh/unknown interval19847ac9
- add token check to raw_open edit view5673fabe
- Better status management.Compare with previous version
added 1 commit
815ba506
- fix for small devicesCompare with previous version
resolved all discussions
merged
mentioned in commit
b49d96d18f
mentioned in issue #65