WIP: Qwann/k fet/kfet open #488
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#488
Loading…
Reference in a new issue
No description provided.
Delete branch "qwann/k-fet/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?
Has to be reviewed after !164
Reassigned to @delobell
Added 5 commits:
dcc19bd9
- article.hidden in article & article_readca17e36b
- clean display for is_sold964dd716
- Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_open8e462134
- home base htmlb7040d58
- adding KFET_FORCE_CLOSE in cacheAdded ~26 label
Hum, question bête : le help_text est commun tous les articles, ou il est dupliqué pour l'ensemble des articles de la DB ?
Indentation ?
Ce n'est pas stocké dans la base de donnée, c'est uniquement écrit ici dans le code python.
C'est juste le texte d'aide par défaut qui sera affiché pour les
ModelForm
sur le champhidden
d'un article. Et cela peut être override au cas par cas dans les formulaires au besoin.Added 4 commits:
392338ed
- indent0f96d3bc
- moving kfet_open urlf8e09cf2
- unknow status updated every 30sce96bc4b
- Status added on home pageAdded 1 commit:
5694e4b9
- migrations mergedAdded 1 commit:
8e7b9e2f
- force close workingAdded 1 commit:
f18bb9f3
- permission addedAdded 1 commit:
cd31c552
- permission workingAdded 10 commits:
k-fet
5ddcd2f1
- Merge branch 'k-fet' into qwann/k-fet/home52dda7be
- Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_opentypo
Il ne vaudrait pas mieux utiliser le builtin
@permission_required
?Il ne faudrait pas un
True
ici ?On utilise déjà Moment.js autre part, ça peut te simplifier la vie.
J'avoue que j'ai un peu plagié ce qui avait été fait avant. @delobell une raison particulière pour avoir fait ça comme ça plutôt qu'autrement ?
Meeeeh, ça se vaut, non ? Une raison particulière de vouloir préférer
True
àFalse
?Added 1 commit:
b0643c09
- typoPour l'instant ton code fait :
if kfet_open == False: kfet_open = False
, ce qui est un peu redondant.Nope,
if not kfet_open:
vérifie sikfet_open
a bien été récupéré dans le cache. Si il n'était pas dans le cache, on l'initialise alors àFalse
:)Awi tiens
Le
teamkfet_required
permet d'avoir plus d'indépendance dans la façon de gérer qui a accès aux interfaces de l'équipe K-Fêt. Imaginons que les cofeux y soient autorisés de base ou que seuls les gens nommés il y a moins de 6 mois en aient me droit (exemples...), il suffit de modifier ça là.Par contre, pour le
can_force_close
, c'est déjà bien spécifique et je pense pas qu'il y ait besoin de sortir un tag spécifique. Plutôt, comme les autres vues (je crois), unteamkfet_required
sur la vue puis utiliser lehas_perm
me semble plus legit.Pour note, on vient de créer une classe
KfetWebSocket
générique surmaster
, donc ça serait pratique de déjà mettre celui là dedans =)added 286 commits
e6f03698
- 1 commit from branchk-fet
fe8f18ff
- Utilise django_custommailb9c2efaf
- Merge branch 'master' into Kerl/use_django_custommail5f29caec
- cleanup29801528
- cleanup et nouvelle implem de custommailb39806e1
- Migration et chargement des emails8bf2f715
- Correction de bugs et nettoyagea5b69019
- Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…74abd6c8
- Fixe le comportement de syncmail9482ab44
- Update provisioningdd60c58b
- Follows the changes in custommailc72393d5
- Update readme3c7558c8
- The end of Clipper01ce9557
- Fixescff4f176
- typocd3faa09
- Merge branch 'master' into Kerl/use_django_custommailc355c71f
- Completion starts only when we have 3 characters0d09bf62
- Completion starts only when we have 3 characters4f66f161
- Rend la page état des demandes/ratios cohérentedc33160e
- Plus d'infos + pluralisation6aa237c4
- Merge branch 'Kerl/fix_choices_count' into 'master'0e7fc99a
- Merge branch 'k-fet' into 'master'3f9f19ef
- Hotfix pour le JS absent1b82b230
- Typo et suppression de la py2 compat540b6f38
- Merge branch 'master' into Kerl/ldap84f9b80f
- On ne met pas root dans la fixture users5fa0618a
- Message pour moldu3e96c982
- Fix urls in productioncef40dff
- Typo et renommage697511a8
- Merge branch 'hotfix/urls' into 'master'993b8ae8
- Merge branch 'master' into Kerl/use_django_custommailb52c0bf6
- Merge branch 'Kerl/csv'6b954033
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF045f3f58
- Metinscription-formset
àa la bonne place30cebac6
- Merge branch 'hotfix/bda-static'c2b74ea6
- Merge branch 'Kerl/bda_models_update' into 'master'cf4a3ec6
- Traitement des messages18b18692
- Dev data loaded using a django admin commandba88b943
- Fixes and cleanup45856ca8
- update READMEd31b9f59
- Traitement des messages0666b528
- Proper use of messages in survies91fff8a1
- Proper use of messages in BdA inscriptionefea92b5
- Proper use of messages in placesb12b5c93
- Proper use of messages in inscription-reventes1fe1b94a
- Proper use of messages in eventsb7ecac3d
- Proper use of messages in profile edit51c0e2da
- proper use of messages in registrationfd42563f
- Proper use of messages in calendar subscription8c34e2f8
- proper use of messages in petits cours54f09820
- Merge branch 'Kerl/messages' of https://git.eleves.ens.fr/cof-geek/GestioCOF into Kerl/messages13da42b8
- typof274c1e9
- Un seul message pour la liste des inscriptions1e5c55a5
- update readmebb4e9dde
- End of py2 support2bc5f3d6
- Style and PEP881681ad0
- Turn 2 functions into class/objects methods9aa4782d
- Move petits-cours forms in another file45eb384c
- Use class-based views217624fe
- Merge branch 'Kerl/fix_dev_data' into 'master'2013fec6
- move fixtures8346dd65
- dev data kfetbb78091c
- checkout fixture066df73b
- add different operation types1062fd48
- update provisioning4e62eb0a
- create checkout dynamicallyc6e84b8c
- bug3b318cf6
- correct timedeltab692b92f
- correct return value for get_or_create5ea97055
- use decimal01dbf729
- withdrawals cost money9ade88d0
- add stdout log46638bd6
- fixes #85ca73dc27
- move template8e7428a1
- Répercute le déplacement du template dans les vuese32faf61
- Merge branch 'Aufinal/select_all' into 'master'f7ec5ef9
- Grise les spectacles passés dans la liste des spectacles pour le BdA399e5ca1
- Jolie mise en page pour les demandes de petit cours80d8cb6b
- Mise à jour des dépendances jquery-ui et jquery839d60e4
- Support tactile jquery-uif0774a3e
- Merge branch 'robin' into 'master'd4b696db
- Merge branch 'master' into Kerl/ldap02a8e74e
- Optimize code, better queries1060a0a3
- Merge branch 'Kerl/ldap' into 'master'ed015084
- using context_object_name and moving template5136e394
- Merge branch 'Kerl/modernize_petitscours'06eafdae
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF6e559057
- Merge branch 'master' into Kerl/use_django_custommail71fb020e
- Merge branch 'master' into Kerl/messages99a19f85
- typo675ff994
- Supprime les doublons avec jquery0e759405
- Change the migration index62dc04c1
- cleaning ugly html5d35112c
- Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…b4bf4374
- Minor fixes27bf59d4
- Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…924a289a
- cleaning UGLY django template styleec7012ee
- Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…5b54d76e
- Merge branch 'Kerl/use_django_custommail'65341f0e
- Minor error70e89e37
- Merge branch 'master' into Kerl/messages313b5cf6
- Merge branch 'master' into Aufinal/dev_data_kfetd41d9053
- PEP8427859c2
- Oublisdc822d3c
- Add license266a789d
- Merge branch 'Kerl/messages'f5b23174
- Fix test_database issue in bootstrap85e593dd
- Removes the test database if the build failse8d21882
- Fix CI config451fddaf
- Fix CI scriptd9402e10
- change resell and reinit formsd524d928
- better annulation check8953d3de
- remove immediate maild96f4ead
- Revert "remove immediate mail"42a93027
- fix pep8 + move templatea0a7be43
- Merge branch 'Aufinal/fix_revente' into 'master'ee6de356
- category addcost addedd0938dfa
- Merge branch 'k-fet' into qwann/k-fet/category_addcostf64a865d
- Côté serveur du catalogue des spectacles BdAc3775437
- Move registration_form to gestioncof/98abe716
- Handleerrors=None
in _finalize_traitementc5137944
- Fix wrong email generation in _traitement_postc4edff71
- Disable the fail_silently option for emails84a39b41
- Fix wrong email generation in _traitement_post (2)e98a4079
- Rename bits -> search_words in kfet.autocomplete3acc8bca
- s/username/clipper in highlight_clipper3793863b
- Merge remote-tracking branch 'origin/k-fet' into Aufinal/merge_k-fet68334e15
- Gestion des citations7ecea0e3
- Meilleure gestion des dates et timezonesa8428f1b
- Précision des tirages qui doivent apparaître dans le catalogue741f0183
- Prevent ldap injections in autocompletion views55b67f38
- Notice the users about the α-numeric restrictionfcc2ab88
- frontend workinga107fa03
- migration renamedde724a2c
- PEP8 for perform_operationf512c1bd
- Add better checkout creation29d76540
- Root kfet account47b7d934
- PEP8 on imports51fba4da
- Log messages4899bba1
- Replace list comprehensions by iterators in dala057869d
- Merge branch 'Aufinal/fix_kfet_devdata' into 'master'dc50a23b
- Escape the content of the regex inhighligh_text
74f0b63e
- Change ldap query behaviour if multiple inputs0356ec34
- Add createopes command36dc6439
- Plug createopes into loadkfetdevdata2c63e6b6
- Use django.contrib.auth decorators6d36d50e
- Fix 'at' property for transfergroups0aed9756
- Fix the empty-query test in autocomplete59f57793
- typoae38b5d1
- Merge branch 'hotfix/prevent_ldap_injection' into 'master'aa247b46
- Fix search box (hackish)dd4d1f30
- Improves creation efficiency5e802217
- Remove lambda operators33b3d558
- Merge branch 'Aufinal/decorators' into 'master'7520effe
- Fix K-Psul freeze when no-team passwordcb132804
- Merge branch 'master' into 'Aufinal/merge_k-fet'a221d784
- Merge branch 'Aufinal/merge_k-fet' into 'master'ff595963
- fix missing bracketf063c347
- Merge branch 'Aufinal/fix_password_freeze' into 'master'cadaf431
- Remove is_checkout field from Operation modeld7740e66
- adapt code to is_checkout removal44fbc928
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/editions608e67fe
- Add RunPython script to migration5f3f0440
- Add revert function to migration79452777
- Add more explicit migration name215d4a63
- Merge branch 'Aufinal/create_opes' into 'master'ae82c340
- add missing migrationbc0affc0
- Remove autocomplete leftoversd8f572bb
- Remove group check in is_checkout14e0d809
- Add euros symbolsf645121f
- Add error when editing LIQ9a081dda
- PEP8 on Operation model06a89055
- Simpler migration946182f1
- Simpler migration revert3b793dc7
- Add first improvements for inventorye6a1d168
- Websocket to manage possible kpsul conflicts579711ed
- Hotfix: typo in template name47e1071b
- Merge branch 'Production'c95d7cb6
- Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOFca6e120a
- Fix ws stock update2d22c202
- Prevent side effects while sending emailsf32f83db
- Merge branch 'hotfix/bda_mail_context'eb7d436b
- Add "count finished" checkbox95c59de4
- Add supplier to dev data56b5fd62
- Always randomize supplier articles31888e33
- simpler jquery selection998838ca
- Add update button63cb7b7f
- Merge branch 'Aufinal/dev_supplier' into 'master'0c212383
- Change misc field when updating079666c0
- Use verbose names and form snippets for article_create8e65298c
- Idem for article_update91d5fe23
- Idem for checkoutstatementsc1b8849c
- Idem for supplier_form20561eb5
- Idem for account_updateebf133d9
- Lowercase model verbose namesc4a3c1a9
- Formatage du code selon PEP-8a9c8de75
- Utilisation des JsonResponsef3b9266e
- Simplification du code avec des méthodes de Django73c21d83
- Correction des exceptions et de l'URL des images8cf14d3f
- Possibilité de filtrer sur plusieurs salles/catégoriese54324e9
- Use col-offset50cb6e51
- Use Set() natively6ac1241b
- Listen to submit instead530aafad
- Use widget_tweaksfdcd2e86
- Use widget_tweaks everywhere499c0815
- Websocket class and instance8a1af73e
- A migration for the new fieldappear_catalogue
67b44219
- PEP8 does not like\
1d8e084a
- websocket update when addcost49c84076
- Manage help_text and errors413df080
- Add url get method8bf1bd53
- Websocket as member38bfccf3
- Change class nameab15dbae
- Add handler managementaa29621c
- Prevents special opes on LIQfafa7e53
- Merge branch 'Aufinal/websockets' into 'master'1286606f
- Merge branch 'Aufinal/hotfix/liq_withdraw' into 'master'92fd94d1
- Merge remote-tracking branch 'origin/master' into Aufinal/fix_stock_ws9c9ad21d
- Add variable for article linecf03fba1
- Remove whitespace by using elif063446ef
- Use columns for authentication and submit8da832c1
- Use nice authentication in orders tooccf5f806
- Merge remote-tracking branch 'origin/master' into Aufinal/editionsc3553160
- Merge branch 'Aufinal/fix_stock_ws' into 'master'271654b4
- No need for intermediate error reportinge20ab2f3
- Use set delete()6797c92a
- Ok ok je me rendsbbb517fb
- Fix article stat display6b8001db
- Correction des messages d'erreur et de la gestion des exceptions66dd7848
- Correction du format de l'URL1eed2283
- Merge branch 'Roussille/bda' into 'master'cce0411e
- Merge both 0048 migrations9cdf0640
- Handle incomplete values from the LDAPa793e9a2
- exclude empty strings from ldap results944a73ee
- Merge remote-tracking branch 'origin/master' into Aufinal/editions91a05787
- Merge migrationse4d51de5
- Merge branch 'hotfix/autocomplete_no_value'55116f71
- Merge branch 'Aufinal/stat_article' into 'master'c9973cde
- Help text for password length96597aa1
- clean some kfet templatese8fdd083
- delete unused class-views78aa5df3
- fix template errorf6022ecf
- Add str to Transfer model87b9db52
- Refactor py base stats and account balance stats1ee993e1
- Add permission check to AccountStatBalancedf7e9353
- Clean GET params in ajax calls48721b7d
- Reduce graph size6ebea655
- Merge branch 'Aufinal/graph_size' into aureplop/fix_stats31261fd3
- set height canvas graph & fix graph displayf5852472
- Refactor Account Operations statsc01de558
- Clean Article statsd97a7be8
- stats: fix begin of balances graphsb3a9ad8a
- clean mixinb157ffac
- Merge branch 'Aufinal/kfet_forms' into 'master'7dc233c0
- Merge branch 'Aufinal/editions' into 'master'1bb83ccd
- simplify StatScale769c3763
- delete debug msg87bc90ec
- Begin/end balance stat graphb113a57b
- Fix update function40da3bc2
- Listen on input903da795
- clean kfet statistic.js6d2e150a
- clean align10d2b58f
- clean some commentsad2c8537
- Add Runpython (oops)5607d7c9
- Migration pourkfet_forms
b3418db3
- Merge branch 'Aufinal/edit_migration' into 'master'f46ba0dd
- Merge branch 'master' into Aufinal/inventaires32474a68
- Don't update input when unchecking51acb4e0
- Use new WS classf13d1072
- Add simple tests for the stat viewsdf467767
- fix default GET param 'types' on operations stats7989a07b
- cleaner fixb383923d
- Merge branch 'Kerl/test_stats' into aureplop/fix_stats0405eee7
- Merge branch 'Aufinal/inventaires' into 'master'278459e8
- typo.......72615bf4
- small fixes885e40fd
- cleaner scalesdc07b072
- Flatten scale args in GET params of stats urlsba11aa49
- categories are updatable73500069
- PEP885ba44c2
- Tests the redirection using the appropriate methodfe66a6ef
- Merge branch 'aureplop/fix_stats' into 'master'42728e58
- Merge branch 'master' into qwann/k-fet/category_addcost3ee9de93
- few improvements on article categoryebf948d0
- Merge branch 'qwann/k-fet/category_addcost' into 'master'a5671fdf
- Merge branch 'master' into qwann/k-fet/kfet_openCompare with previous version
added 1 commit
ef8fec89
- migration renamedCompare with previous version
changed target branch from
k-fet
tomaster
Tu peux retirer ces trois lignes:
strict_ordering
est par défaut àFalse
slight_ordering
n'existe pas@narmanli du coup je reste sur l'avis que ce décorateur n'est pas nécessaire
Tu peux aussi décorer dans l'URL pour faire
permission_required('can_force_close')
Parmi les nouveaux imports, seulement
View
semble être utilisé.C'est possible d'avoir des explications sur qu'est-ce qui fait quoi ?
A quoi servent les valeurs en cache et l'objectif de chaque url avec les paramètres qu'elle accepte stp ?
added 12 commits
master
531a4611
- Merge branch 'master' into qwann/k-fet/kfet_opendeb0d4de
- moving migration againCompare with previous version
added 1 commit
e18c2c69
- new migrationCompare with previous version
Ils sont prévus pour rester ces logs ?
Idem
Deux petites remarques esthétiques :
Oui :p
added 1 commit
15873085
- small fixesCompare with previous version
added 1 commit
5c6a73c5
- kfet_open uses moment.jsCompare with previous version
added 1 commit
cb9ba76a
- small fixesCompare with previous version
added 1 commit
e0b0a531
- stupidness removedCompare with previous version
Ça donne un peu l'impression que cette fonction sert à fermer la K-Fêt, et pas à récupérer son état...
added 1 commit
be8d249e
- remove useless codeCompare with previous version
Il y a
cache.get_or_set
qui fait exactement ce que tu fais ici.mentioned in merge request !221
KfetOpen
pour le calquer sur celuiKPsul
(il va devenir très court :-P, et pas besoin d'user PermMixin à mon avis).kfet_config
pour avoir une persistance. Rien de critique à ça cependant...Pour gérer le second et le dernier point, on peut imaginer que
KFetStatus.is_open
donne l'information en prenant en compte l'état deforce_close
etKFetStatus.raw_open
donne la vraie information. Pour le dernier point, par exemple, il faudrait un second WS pour les personnes ayant suffisamment de permissions qui leur permettraient de connaître les statuts deraw_open
etforce_close
, tandis que le premier WS est accessible par tous mais ne donne que leis_open
.mentioned in issue #65
resolved all discussions
Pas en 1.8 :(
resolved all discussions
typo: toutes
Je sais que ce genre de trucs traine à d'autres endroits mais il n'y a pas un moyen plus générique de gérer ça ?
On a géré ça avec @delobell dans une classe générale
KfetWebSocket
(je crois), donc il faut en hériter pour éviter ce bout de code.J'ai pas un avis très prononcé là dessus mais j'ai tendance à penser que les pages destinées aux script devraient plutôt être du JSON (facile à parser) qu'en texte brut/HTML (pour les humains). Et il me semble que c’en est une, je me trompe ?
Je parle des
/gestion
au début des urls, pardon ce n'était pas très clair.Yup, j'ai vu, je suis en train de m'en occuper :)
Pour le coup c'est essentiellement du debug et déstiné à des humain, c'est pas du tout destiné au script. Mais sinon je suis effectivement très d'accord sur le fait qu'il vaut mieux rendre du JSON aux scripts :)
Ok mais si c'est du debug, c'est amené à rester en prod ?
Euh, comme vous voulez. Pour le coup je n'ai pas d'avis sur la question. La seule personne qui sera amenée à lire ce message c'est le raspi ^^
Du coup laisse
mentioned in merge request !239
merged