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

WIP: Qwann/k fet/kfet open #488

Merged
narmanli merged 0 commits from qwann/k-fet/kfet_open into master 2017-06-23 00:29:16 +02:00
Conversation 63 Commits - Files changed -
narmanli commented 2017-03-05 20:17:31 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Has to be reviewed after !164

Has to be reviewed after !164
narmanli commented 2017-03-05 23:06:51 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Reassigned to @delobell

Reassigned to @delobell
narmanli commented 2017-03-06 02:24:19 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 5 commits:

  • dcc19bd9 - article.hidden in article & article_read
  • ca17e36b - clean display for is_sold
  • 964dd716 - Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_open
  • 8e462134 - home base html
  • b7040d58 - adding KFET_FORCE_CLOSE in cache
Added 5 commits: * dcc19bd9 - article.hidden in article & article_read * ca17e36b - clean display for is_sold * 964dd716 - Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_open * 8e462134 - home base html * b7040d58 - adding KFET_FORCE_CLOSE in cache
narmanli commented 2017-03-06 02:25:44 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added ~26 label

Added ~26 label
manet commented 2017-03-06 08:52:28 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Hum, question bête : le help_text est commun tous les articles, ou il est dupliqué pour l'ensemble des articles de la DB ?

Hum, question bête : le help_text est commun tous les articles, ou il est dupliqué pour l'ensemble des articles de la DB ?
manet commented 2017-03-06 08:55:14 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Indentation ?

Indentation ?
mpepin commented 2017-03-06 11:40:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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 champ hidden d'un article. Et cela peut être override au cas par cas dans les formulaires au besoin.

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 champ `hidden` d'un article. Et cela peut être override au cas par cas dans les formulaires au besoin.
narmanli commented 2017-03-07 20:21:30 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 4 commits:

Added 4 commits: * 392338ed - indent * 0f96d3bc - moving kfet_open url * f8e09cf2 - unknow status updated every 30s * ce96bc4b - Status added on home page
narmanli commented 2017-03-09 11:10:41 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 5694e4b9 - migrations merged
narmanli commented 2017-03-09 15:04:47 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 8e7b9e2f - force close working
narmanli commented 2017-03-09 17:27:15 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * f18bb9f3 - permission added
narmanli commented 2017-03-10 16:39:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * cd31c552 - permission working
narmanli commented 2017-03-10 16:57:59 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 10 commits:

  • cd31c552...fbd0d760 - 8 commits from branch k-fet
  • 5ddcd2f1 - Merge branch 'k-fet' into qwann/k-fet/home
  • 52dda7be - Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_open
Added 10 commits: * cd31c552...fbd0d760 - 8 commits from branch `k-fet` * 5ddcd2f1 - Merge branch 'k-fet' into qwann/k-fet/home * 52dda7be - Merge branch 'qwann/k-fet/home' into qwann/k-fet/kfet_open
lstephan commented 2017-03-10 23:01:23 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

typo

typo
lstephan commented 2017-03-10 23:03:02 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il ne vaudrait pas mieux utiliser le builtin @permission_required ?

Il ne vaudrait pas mieux utiliser le builtin `@permission_required` ?
lstephan commented 2017-03-10 23:07:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il ne faudrait pas un True ici ?

Il ne faudrait pas un `True` ici ?
lstephan commented 2017-03-10 23:12:07 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

On utilise déjà Moment.js autre part, ça peut te simplifier la vie.

On utilise déjà [Moment.js](https://momentjs.com/docs/) autre part, ça peut te simplifier la vie.
narmanli commented 2017-03-11 01:55:35 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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 ?

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 ?
narmanli commented 2017-03-11 01:57:14 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Meeeeh, ça se vaut, non ? Une raison particulière de vouloir préférer True à False ?

Meeeeh, ça se vaut, non ? Une raison particulière de vouloir préférer `True` à `False` ?
narmanli commented 2017-03-11 02:03:45 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * b0643c09 - typo
lstephan commented 2017-03-11 02:11:48 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Pour l'instant ton code fait : if kfet_open == False: kfet_open = False, ce qui est un peu redondant.

Pour l'instant ton code fait : `if kfet_open == False: kfet_open = False`, ce qui est un peu redondant.
narmanli commented 2017-03-11 02:19:33 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Nope, if not kfet_open: vérifie si kfet_open a bien été récupéré dans le cache. Si il n'était pas dans le cache, on l'initialise alors à False :)

Nope, `if not kfet_open:` vérifie si `kfet_open` a bien été récupéré dans le cache. Si il n'était pas dans le cache, on l'initialise alors à `False` :)
lstephan commented 2017-03-11 02:21:03 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Awi tiens

Awi tiens
delobell commented 2017-03-12 03:10:59 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

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), un teamkfet_required sur la vue puis utiliser le has_permme semble plus legit.

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), un `teamkfet_required` sur la vue puis utiliser le `has_perm`me semble plus legit.
lstephan commented 2017-04-01 04:37:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pour note, on vient de créer une classe KfetWebSocket générique sur master, donc ça serait pratique de déjà mettre celui là dedans =)

Pour note, on vient de créer une classe `KfetWebSocket` générique sur `master`, donc ça serait pratique de déjà mettre celui là dedans =)
narmanli commented 2017-04-05 17:38:52 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 286 commits

  • e6f03698 - 1 commit from branch k-fet
  • fe8f18ff - Utilise django_custommail
  • b9c2efaf - Merge branch 'master' into Kerl/use_django_custommail
  • 5f29caec - cleanup
  • 29801528 - cleanup et nouvelle implem de custommail
  • b39806e1 - Migration et chargement des emails
  • 8bf2f715 - Correction de bugs et nettoyage
  • a5b69019 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
  • 74abd6c8 - Fixe le comportement de syncmail
  • 9482ab44 - Update provisioning
  • dd60c58b - Follows the changes in custommail
  • c72393d5 - Update readme
  • 3c7558c8 - The end of Clipper
  • 01ce9557 - Fixes
  • cff4f176 - typo
  • cd3faa09 - Merge branch 'master' into Kerl/use_django_custommail
  • c355c71f - Completion starts only when we have 3 characters
  • 0d09bf62 - Completion starts only when we have 3 characters
  • 4f66f161 - Rend la page état des demandes/ratios cohérente
  • dc33160e - Plus d'infos + pluralisation
  • 6aa237c4 - Merge branch 'Kerl/fix_choices_count' into 'master'
  • 0e7fc99a - Merge branch 'k-fet' into 'master'
  • 3f9f19ef - Hotfix pour le JS absent
  • 1b82b230 - Typo et suppression de la py2 compat
  • 540b6f38 - Merge branch 'master' into Kerl/ldap
  • 84f9b80f - On ne met pas root dans la fixture users
  • 5fa0618a - Message pour moldu
  • 3e96c982 - Fix urls in production
  • cef40dff - Typo et renommage
  • 697511a8 - Merge branch 'hotfix/urls' into 'master'
  • 993b8ae8 - Merge branch 'master' into Kerl/use_django_custommail
  • b52c0bf6 - Merge branch 'Kerl/csv'
  • 6b954033 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF
  • 045f3f58 - Met inscription-formset àa la bonne place
  • 30cebac6 - Merge branch 'hotfix/bda-static'
  • c2b74ea6 - Merge branch 'Kerl/bda_models_update' into 'master'
  • cf4a3ec6 - Traitement des messages
  • 18b18692 - Dev data loaded using a django admin command
  • ba88b943 - Fixes and cleanup
  • 45856ca8 - update README
  • d31b9f59 - Traitement des messages
  • 0666b528 - Proper use of messages in survies
  • 91fff8a1 - Proper use of messages in BdA inscription
  • efea92b5 - Proper use of messages in places
  • b12b5c93 - Proper use of messages in inscription-reventes
  • 1fe1b94a - Proper use of messages in events
  • b7ecac3d - Proper use of messages in profile edit
  • 51c0e2da - proper use of messages in registration
  • fd42563f - Proper use of messages in calendar subscription
  • 8c34e2f8 - proper use of messages in petits cours
  • 54f09820 - Merge branch 'Kerl/messages' of https://git.eleves.ens.fr/cof-geek/GestioCOF into Kerl/messages
  • 13da42b8 - typo
  • f274c1e9 - Un seul message pour la liste des inscriptions
  • 1e5c55a5 - update readme
  • bb4e9dde - End of py2 support
  • 2bc5f3d6 - Style and PEP8
  • 81681ad0 - Turn 2 functions into class/objects methods
  • 9aa4782d - Move petits-cours forms in another file
  • 45eb384c - Use class-based views
  • 217624fe - Merge branch 'Kerl/fix_dev_data' into 'master'
  • 2013fec6 - move fixtures
  • 8346dd65 - dev data kfet
  • bb78091c - checkout fixture
  • 066df73b - add different operation types
  • 1062fd48 - update provisioning
  • 4e62eb0a - create checkout dynamically
  • c6e84b8c - bug
  • 3b318cf6 - correct timedelta
  • b692b92f - correct return value for get_or_create
  • 5ea97055 - use decimal
  • 01dbf729 - withdrawals cost money
  • 9ade88d0 - add stdout log
  • 46638bd6 - fixes #85
  • ca73dc27 - move template
  • 8e7428a1 - Répercute le déplacement du template dans les vues
  • e32faf61 - Merge branch 'Aufinal/select_all' into 'master'
  • f7ec5ef9 - Grise les spectacles passés dans la liste des spectacles pour le BdA
  • 399e5ca1 - Jolie mise en page pour les demandes de petit cours
  • 80d8cb6b - Mise à jour des dépendances jquery-ui et jquery
  • 839d60e4 - Support tactile jquery-ui
  • f0774a3e - Merge branch 'robin' into 'master'
  • d4b696db - Merge branch 'master' into Kerl/ldap
  • 02a8e74e - Optimize code, better queries
  • 1060a0a3 - Merge branch 'Kerl/ldap' into 'master'
  • ed015084 - using context_object_name and moving template
  • 5136e394 - Merge branch 'Kerl/modernize_petitscours'
  • 06eafdae - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF
  • 6e559057 - Merge branch 'master' into Kerl/use_django_custommail
  • 71fb020e - Merge branch 'master' into Kerl/messages
  • 99a19f85 - typo
  • 675ff994 - Supprime les doublons avec jquery
  • 0e759405 - Change the migration index
  • 62dc04c1 - cleaning ugly html
  • 5d35112c - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
  • b4bf4374 - Minor fixes
  • 27bf59d4 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
  • 924a289a - cleaning UGLY django template style
  • ec7012ee - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO…
  • 5b54d76e - Merge branch 'Kerl/use_django_custommail'
  • 65341f0e - Minor error
  • 70e89e37 - Merge branch 'master' into Kerl/messages
  • 313b5cf6 - Merge branch 'master' into Aufinal/dev_data_kfet
  • d41d9053 - PEP8
  • 427859c2 - Oublis
  • dc822d3c - Add license
  • 266a789d - Merge branch 'Kerl/messages'
  • f5b23174 - Fix test_database issue in bootstrap
  • 85e593dd - Removes the test database if the build fails
  • e8d21882 - Fix CI config
  • 451fddaf - Fix CI script
  • d9402e10 - change resell and reinit forms
  • d524d928 - better annulation check
  • 8953d3de - remove immediate mail
  • d96f4ead - Revert "remove immediate mail"
  • 42a93027 - fix pep8 + move template
  • a0a7be43 - Merge branch 'Aufinal/fix_revente' into 'master'
  • ee6de356 - category addcost added
  • d0938dfa - Merge branch 'k-fet' into qwann/k-fet/category_addcost
  • f64a865d - Côté serveur du catalogue des spectacles BdA
  • c3775437 - Move registration_form to gestioncof/
  • 98abe716 - Handle errors=None in _finalize_traitement
  • c5137944 - Fix wrong email generation in _traitement_post
  • c4edff71 - Disable the fail_silently option for emails
  • 84a39b41 - Fix wrong email generation in _traitement_post (2)
  • e98a4079 - Rename bits -> search_words in kfet.autocomplete
  • 3acc8bca - s/username/clipper in highlight_clipper
  • 3793863b - Merge remote-tracking branch 'origin/k-fet' into Aufinal/merge_k-fet
  • 68334e15 - Gestion des citations
  • 7ecea0e3 - Meilleure gestion des dates et timezones
  • a8428f1b - Précision des tirages qui doivent apparaître dans le catalogue
  • 741f0183 - Prevent ldap injections in autocompletion views
  • 55b67f38 - Notice the users about the α-numeric restriction
  • fcc2ab88 - frontend working
  • a107fa03 - migration renamed
  • de724a2c - PEP8 for perform_operation
  • f512c1bd - Add better checkout creation
  • 29d76540 - Root kfet account
  • 47b7d934 - PEP8 on imports
  • 51fba4da - Log messages
  • 4899bba1 - Replace list comprehensions by iterators in dal
  • a057869d - Merge branch 'Aufinal/fix_kfet_devdata' into 'master'
  • dc50a23b - Escape the content of the regex in highligh_text
  • 74f0b63e - Change ldap query behaviour if multiple inputs
  • 0356ec34 - Add createopes command
  • 36dc6439 - Plug createopes into loadkfetdevdata
  • 2c63e6b6 - Use django.contrib.auth decorators
  • 6d36d50e - Fix 'at' property for transfergroups
  • 0aed9756 - Fix the empty-query test in autocomplete
  • 59f57793 - typo
  • ae38b5d1 - Merge branch 'hotfix/prevent_ldap_injection' into 'master'
  • aa247b46 - Fix search box (hackish)
  • dd4d1f30 - Improves creation efficiency
  • 5e802217 - Remove lambda operators
  • 33b3d558 - Merge branch 'Aufinal/decorators' into 'master'
  • 7520effe - Fix K-Psul freeze when no-team password
  • cb132804 - Merge branch 'master' into 'Aufinal/merge_k-fet'
  • a221d784 - Merge branch 'Aufinal/merge_k-fet' into 'master'
  • ff595963 - fix missing bracket
  • f063c347 - Merge branch 'Aufinal/fix_password_freeze' into 'master'
  • cadaf431 - Remove is_checkout field from Operation model
  • d7740e66 - adapt code to is_checkout removal
  • 44fbc928 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/editions
  • 608e67fe - Add RunPython script to migration
  • 5f3f0440 - Add revert function to migration
  • 79452777 - Add more explicit migration name
  • 215d4a63 - Merge branch 'Aufinal/create_opes' into 'master'
  • ae82c340 - add missing migration
  • bc0affc0 - Remove autocomplete leftovers
  • d8f572bb - Remove group check in is_checkout
  • 14e0d809 - Add euros symbols
  • f645121f - Add error when editing LIQ
  • 9a081dda - PEP8 on Operation model
  • 06a89055 - Simpler migration
  • 946182f1 - Simpler migration revert
  • 3b793dc7 - Add first improvements for inventory
  • e6a1d168 - Websocket to manage possible kpsul conflicts
  • 579711ed - Hotfix: typo in template name
  • 47e1071b - Merge branch 'Production'
  • c95d7cb6 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF
  • ca6e120a - Fix ws stock update
  • 2d22c202 - Prevent side effects while sending emails
  • f32f83db - Merge branch 'hotfix/bda_mail_context'
  • eb7d436b - Add "count finished" checkbox
  • 95c59de4 - Add supplier to dev data
  • 56b5fd62 - Always randomize supplier articles
  • 31888e33 - simpler jquery selection
  • 998838ca - Add update button
  • 63cb7b7f - Merge branch 'Aufinal/dev_supplier' into 'master'
  • 0c212383 - Change misc field when updating
  • 079666c0 - Use verbose names and form snippets for article_create
  • 8e65298c - Idem for article_update
  • 91d5fe23 - Idem for checkoutstatements
  • c1b8849c - Idem for supplier_form
  • 20561eb5 - Idem for account_update
  • ebf133d9 - Lowercase model verbose names
  • c4a3c1a9 - Formatage du code selon PEP-8
  • a9c8de75 - Utilisation des JsonResponse
  • f3b9266e - Simplification du code avec des méthodes de Django
  • 73c21d83 - Correction des exceptions et de l'URL des images
  • 8cf14d3f - Possibilité de filtrer sur plusieurs salles/catégories
  • e54324e9 - Use col-offset
  • 50cb6e51 - Use Set() natively
  • 6ac1241b - Listen to submit instead
  • 530aafad - Use widget_tweaks
  • fdcd2e86 - Use widget_tweaks everywhere
  • 499c0815 - Websocket class and instance
  • 8a1af73e - A migration for the new field appear_catalogue
  • 67b44219 - PEP8 does not like \
  • 1d8e084a - websocket update when addcost
  • 49c84076 - Manage help_text and errors
  • 413df080 - Add url get method
  • 8bf1bd53 - Websocket as member
  • 38bfccf3 - Change class name
  • ab15dbae - Add handler management
  • aa29621c - Prevents special opes on LIQ
  • fafa7e53 - 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_ws
  • 9c9ad21d - Add variable for article line
  • cf03fba1 - Remove whitespace by using elif
  • 063446ef - Use columns for authentication and submit
  • 8da832c1 - Use nice authentication in orders too
  • ccf5f806 - Merge remote-tracking branch 'origin/master' into Aufinal/editions
  • c3553160 - Merge branch 'Aufinal/fix_stock_ws' into 'master'
  • 271654b4 - No need for intermediate error reporting
  • e20ab2f3 - Use set delete()
  • 6797c92a - Ok ok je me rends
  • bbb517fb - Fix article stat display
  • 6b8001db - Correction des messages d'erreur et de la gestion des exceptions
  • 66dd7848 - Correction du format de l'URL
  • 1eed2283 - Merge branch 'Roussille/bda' into 'master'
  • cce0411e - Merge both 0048 migrations
  • 9cdf0640 - Handle incomplete values from the LDAP
  • a793e9a2 - exclude empty strings from ldap results
  • 944a73ee - Merge remote-tracking branch 'origin/master' into Aufinal/editions
  • 91a05787 - Merge migrations
  • e4d51de5 - Merge branch 'hotfix/autocomplete_no_value'
  • 55116f71 - Merge branch 'Aufinal/stat_article' into 'master'
  • c9973cde - Help text for password length
  • 96597aa1 - clean some kfet templates
  • e8fdd083 - delete unused class-views
  • 78aa5df3 - fix template error
  • f6022ecf - Add str to Transfer model
  • 87b9db52 - Refactor py base stats and account balance stats
  • 1ee993e1 - Add permission check to AccountStatBalance
  • df7e9353 - Clean GET params in ajax calls
  • 48721b7d - Reduce graph size
  • 6ebea655 - Merge branch 'Aufinal/graph_size' into aureplop/fix_stats
  • 31261fd3 - set height canvas graph & fix graph display
  • f5852472 - Refactor Account Operations stats
  • c01de558 - Clean Article stats
  • d97a7be8 - stats: fix begin of balances graphs
  • b3a9ad8a - clean mixin
  • b157ffac - Merge branch 'Aufinal/kfet_forms' into 'master'
  • 7dc233c0 - Merge branch 'Aufinal/editions' into 'master'
  • 1bb83ccd - simplify StatScale
  • 769c3763 - delete debug msg
  • 87bc90ec - Begin/end balance stat graph
  • b113a57b - Fix update function
  • 40da3bc2 - Listen on input
  • 903da795 - clean kfet statistic.js
  • 6d2e150a - clean align
  • 10d2b58f - clean some comments
  • ad2c8537 - Add Runpython (oops)
  • 5607d7c9 - Migration pour kfet_forms
  • b3418db3 - Merge branch 'Aufinal/edit_migration' into 'master'
  • f46ba0dd - Merge branch 'master' into Aufinal/inventaires
  • 32474a68 - Don't update input when unchecking
  • 51acb4e0 - Use new WS class
  • f13d1072 - Add simple tests for the stat views
  • df467767 - fix default GET param 'types' on operations stats
  • 7989a07b - cleaner fix
  • b383923d - Merge branch 'Kerl/test_stats' into aureplop/fix_stats
  • 0405eee7 - Merge branch 'Aufinal/inventaires' into 'master'
  • 278459e8 - typo.......
  • 72615bf4 - small fixes
  • 885e40fd - cleaner scales
  • dc07b072 - Flatten scale args in GET params of stats urls
  • ba11aa49 - categories are updatable
  • 73500069 - PEP8
  • 85ba44c2 - Tests the redirection using the appropriate method
  • fe66a6ef - Merge branch 'aureplop/fix_stats' into 'master'
  • 42728e58 - Merge branch 'master' into qwann/k-fet/category_addcost
  • 3ee9de93 - few improvements on article category
  • ebf948d0 - Merge branch 'qwann/k-fet/category_addcost' into 'master'
  • a5671fdf - Merge branch 'master' into qwann/k-fet/kfet_open

Compare with previous version

added 286 commits * e6f03698 - 1 commit from branch `k-fet` * fe8f18ff - Utilise django_custommail * b9c2efaf - Merge branch 'master' into Kerl/use_django_custommail * 5f29caec - cleanup * 29801528 - cleanup et nouvelle implem de custommail * b39806e1 - Migration et chargement des emails * 8bf2f715 - Correction de bugs et nettoyage * a5b69019 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO… * 74abd6c8 - Fixe le comportement de syncmail * 9482ab44 - Update provisioning * dd60c58b - Follows the changes in custommail * c72393d5 - Update readme * 3c7558c8 - The end of Clipper * 01ce9557 - Fixes * cff4f176 - typo * cd3faa09 - Merge branch 'master' into Kerl/use_django_custommail * c355c71f - Completion starts only when we have 3 characters * 0d09bf62 - Completion starts only when we have 3 characters * 4f66f161 - Rend la page état des demandes/ratios cohérente * dc33160e - Plus d'infos + pluralisation * 6aa237c4 - Merge branch 'Kerl/fix_choices_count' into 'master' * 0e7fc99a - Merge branch 'k-fet' into 'master' * 3f9f19ef - Hotfix pour le JS absent * 1b82b230 - Typo et suppression de la py2 compat * 540b6f38 - Merge branch 'master' into Kerl/ldap * 84f9b80f - On ne met pas root dans la fixture users * 5fa0618a - Message pour moldu * 3e96c982 - Fix urls in production * cef40dff - Typo et renommage * 697511a8 - Merge branch 'hotfix/urls' into 'master' * 993b8ae8 - Merge branch 'master' into Kerl/use_django_custommail * b52c0bf6 - Merge branch 'Kerl/csv' * 6b954033 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF * 045f3f58 - Met `inscription-formset` àa la bonne place * 30cebac6 - Merge branch 'hotfix/bda-static' * c2b74ea6 - Merge branch 'Kerl/bda_models_update' into 'master' * cf4a3ec6 - Traitement des messages * 18b18692 - Dev data loaded using a django admin command * ba88b943 - Fixes and cleanup * 45856ca8 - update README * d31b9f59 - Traitement des messages * 0666b528 - Proper use of messages in survies * 91fff8a1 - Proper use of messages in BdA inscription * efea92b5 - Proper use of messages in places * b12b5c93 - Proper use of messages in inscription-reventes * 1fe1b94a - Proper use of messages in events * b7ecac3d - Proper use of messages in profile edit * 51c0e2da - proper use of messages in registration * fd42563f - Proper use of messages in calendar subscription * 8c34e2f8 - proper use of messages in petits cours * 54f09820 - Merge branch 'Kerl/messages' of https://git.eleves.ens.fr/cof-geek/GestioCOF into Kerl/messages * 13da42b8 - typo * f274c1e9 - Un seul message pour la liste des inscriptions * 1e5c55a5 - update readme * bb4e9dde - End of py2 support * 2bc5f3d6 - Style and PEP8 * 81681ad0 - Turn 2 functions into class/objects methods * 9aa4782d - Move petits-cours forms in another file * 45eb384c - Use class-based views * 217624fe - Merge branch 'Kerl/fix_dev_data' into 'master' * 2013fec6 - move fixtures * 8346dd65 - dev data kfet * bb78091c - checkout fixture * 066df73b - add different operation types * 1062fd48 - update provisioning * 4e62eb0a - create checkout dynamically * c6e84b8c - bug * 3b318cf6 - correct timedelta * b692b92f - correct return value for get_or_create * 5ea97055 - use decimal * 01dbf729 - withdrawals cost money * 9ade88d0 - add stdout log * 46638bd6 - fixes #85 * ca73dc27 - move template * 8e7428a1 - Répercute le déplacement du template dans les vues * e32faf61 - Merge branch 'Aufinal/select_all' into 'master' * f7ec5ef9 - Grise les spectacles passés dans la liste des spectacles pour le BdA * 399e5ca1 - Jolie mise en page pour les demandes de petit cours * 80d8cb6b - Mise à jour des dépendances jquery-ui et jquery * 839d60e4 - Support tactile jquery-ui * f0774a3e - Merge branch 'robin' into 'master' * d4b696db - Merge branch 'master' into Kerl/ldap * 02a8e74e - Optimize code, better queries * 1060a0a3 - Merge branch 'Kerl/ldap' into 'master' * ed015084 - using context_object_name and moving template * 5136e394 - Merge branch 'Kerl/modernize_petitscours' * 06eafdae - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF * 6e559057 - Merge branch 'master' into Kerl/use_django_custommail * 71fb020e - Merge branch 'master' into Kerl/messages * 99a19f85 - typo * 675ff994 - Supprime les doublons avec jquery * 0e759405 - Change the migration index * 62dc04c1 - cleaning ugly html * 5d35112c - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO… * b4bf4374 - Minor fixes * 27bf59d4 - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO… * 924a289a - cleaning UGLY django template style * ec7012ee - Merge branch 'Kerl/use_django_custommail' of git.eleves.ens.fr:cof-geek/gestioCO… * 5b54d76e - Merge branch 'Kerl/use_django_custommail' * 65341f0e - Minor error * 70e89e37 - Merge branch 'master' into Kerl/messages * 313b5cf6 - Merge branch 'master' into Aufinal/dev_data_kfet * d41d9053 - PEP8 * 427859c2 - Oublis * dc822d3c - Add license * 266a789d - Merge branch 'Kerl/messages' * f5b23174 - Fix test_database issue in bootstrap * 85e593dd - Removes the test database if the build fails * e8d21882 - Fix CI config * 451fddaf - Fix CI script * d9402e10 - change resell and reinit forms * d524d928 - better annulation check * 8953d3de - remove immediate mail * d96f4ead - Revert "remove immediate mail" * 42a93027 - fix pep8 + move template * a0a7be43 - Merge branch 'Aufinal/fix_revente' into 'master' * ee6de356 - category addcost added * d0938dfa - Merge branch 'k-fet' into qwann/k-fet/category_addcost * f64a865d - Côté serveur du catalogue des spectacles BdA * c3775437 - Move registration_form to gestioncof/ * 98abe716 - Handle `errors=None` in _finalize_traitement * c5137944 - Fix wrong email generation in _traitement_post * c4edff71 - Disable the fail_silently option for emails * 84a39b41 - Fix wrong email generation in _traitement_post (2) * e98a4079 - Rename bits -> search_words in kfet.autocomplete * 3acc8bca - s/username/clipper in highlight_clipper * 3793863b - Merge remote-tracking branch 'origin/k-fet' into Aufinal/merge_k-fet * 68334e15 - Gestion des citations * 7ecea0e3 - Meilleure gestion des dates et timezones * a8428f1b - Précision des tirages qui doivent apparaître dans le catalogue * 741f0183 - Prevent ldap injections in autocompletion views * 55b67f38 - Notice the users about the α-numeric restriction * fcc2ab88 - frontend working * a107fa03 - migration renamed * de724a2c - PEP8 for perform_operation * f512c1bd - Add better checkout creation * 29d76540 - Root kfet account * 47b7d934 - PEP8 on imports * 51fba4da - Log messages * 4899bba1 - Replace list comprehensions by iterators in dal * a057869d - Merge branch 'Aufinal/fix_kfet_devdata' into 'master' * dc50a23b - Escape the content of the regex in `highligh_text` * 74f0b63e - Change ldap query behaviour if multiple inputs * 0356ec34 - Add createopes command * 36dc6439 - Plug createopes into loadkfetdevdata * 2c63e6b6 - Use django.contrib.auth decorators * 6d36d50e - Fix 'at' property for transfergroups * 0aed9756 - Fix the empty-query test in autocomplete * 59f57793 - typo * ae38b5d1 - Merge branch 'hotfix/prevent_ldap_injection' into 'master' * aa247b46 - Fix search box (hackish) * dd4d1f30 - Improves creation efficiency * 5e802217 - Remove lambda operators * 33b3d558 - Merge branch 'Aufinal/decorators' into 'master' * 7520effe - Fix K-Psul freeze when no-team password * cb132804 - Merge branch 'master' into 'Aufinal/merge_k-fet' * a221d784 - Merge branch 'Aufinal/merge_k-fet' into 'master' * ff595963 - fix missing bracket * f063c347 - Merge branch 'Aufinal/fix_password_freeze' into 'master' * cadaf431 - Remove is_checkout field from Operation model * d7740e66 - adapt code to is_checkout removal * 44fbc928 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF into Aufinal/editions * 608e67fe - Add RunPython script to migration * 5f3f0440 - Add revert function to migration * 79452777 - Add more explicit migration name * 215d4a63 - Merge branch 'Aufinal/create_opes' into 'master' * ae82c340 - add missing migration * bc0affc0 - Remove autocomplete leftovers * d8f572bb - Remove group check in is_checkout * 14e0d809 - Add euros symbols * f645121f - Add error when editing LIQ * 9a081dda - PEP8 on Operation model * 06a89055 - Simpler migration * 946182f1 - Simpler migration revert * 3b793dc7 - Add first improvements for inventory * e6a1d168 - Websocket to manage possible kpsul conflicts * 579711ed - Hotfix: typo in template name * 47e1071b - Merge branch 'Production' * c95d7cb6 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/gestioCOF * ca6e120a - Fix ws stock update * 2d22c202 - Prevent side effects while sending emails * f32f83db - Merge branch 'hotfix/bda_mail_context' * eb7d436b - Add "count finished" checkbox * 95c59de4 - Add supplier to dev data * 56b5fd62 - Always randomize supplier articles * 31888e33 - simpler jquery selection * 998838ca - Add update button * 63cb7b7f - Merge branch 'Aufinal/dev_supplier' into 'master' * 0c212383 - Change misc field when updating * 079666c0 - Use verbose names and form snippets for article_create * 8e65298c - Idem for article_update * 91d5fe23 - Idem for checkoutstatements * c1b8849c - Idem for supplier_form * 20561eb5 - Idem for account_update * ebf133d9 - Lowercase model verbose names * c4a3c1a9 - Formatage du code selon PEP-8 * a9c8de75 - Utilisation des JsonResponse * f3b9266e - Simplification du code avec des méthodes de Django * 73c21d83 - Correction des exceptions et de l'URL des images * 8cf14d3f - Possibilité de filtrer sur plusieurs salles/catégories * e54324e9 - Use col-offset * 50cb6e51 - Use Set() natively * 6ac1241b - Listen to submit instead * 530aafad - Use widget_tweaks * fdcd2e86 - Use widget_tweaks everywhere * 499c0815 - Websocket class and instance * 8a1af73e - A migration for the new field `appear_catalogue` * 67b44219 - PEP8 does not like `\` * 1d8e084a - websocket update when addcost * 49c84076 - Manage help_text and errors * 413df080 - Add url get method * 8bf1bd53 - Websocket as member * 38bfccf3 - Change class name * ab15dbae - Add handler management * aa29621c - Prevents special opes on LIQ * fafa7e53 - 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_ws * 9c9ad21d - Add variable for article line * cf03fba1 - Remove whitespace by using elif * 063446ef - Use columns for authentication and submit * 8da832c1 - Use nice authentication in orders too * ccf5f806 - Merge remote-tracking branch 'origin/master' into Aufinal/editions * c3553160 - Merge branch 'Aufinal/fix_stock_ws' into 'master' * 271654b4 - No need for intermediate error reporting * e20ab2f3 - Use set delete() * 6797c92a - Ok ok je me rends * bbb517fb - Fix article stat display * 6b8001db - Correction des messages d'erreur et de la gestion des exceptions * 66dd7848 - Correction du format de l'URL * 1eed2283 - Merge branch 'Roussille/bda' into 'master' * cce0411e - Merge both 0048 migrations * 9cdf0640 - Handle incomplete values from the LDAP * a793e9a2 - exclude empty strings from ldap results * 944a73ee - Merge remote-tracking branch 'origin/master' into Aufinal/editions * 91a05787 - Merge migrations * e4d51de5 - Merge branch 'hotfix/autocomplete_no_value' * 55116f71 - Merge branch 'Aufinal/stat_article' into 'master' * c9973cde - Help text for password length * 96597aa1 - clean some kfet templates * e8fdd083 - delete unused class-views * 78aa5df3 - fix template error * f6022ecf - Add str to Transfer model * 87b9db52 - Refactor py base stats and account balance stats * 1ee993e1 - Add permission check to AccountStatBalance * df7e9353 - Clean GET params in ajax calls * 48721b7d - Reduce graph size * 6ebea655 - Merge branch 'Aufinal/graph_size' into aureplop/fix_stats * 31261fd3 - set height canvas graph & fix graph display * f5852472 - Refactor Account Operations stats * c01de558 - Clean Article stats * d97a7be8 - stats: fix begin of balances graphs * b3a9ad8a - clean mixin * b157ffac - Merge branch 'Aufinal/kfet_forms' into 'master' * 7dc233c0 - Merge branch 'Aufinal/editions' into 'master' * 1bb83ccd - simplify StatScale * 769c3763 - delete debug msg * 87bc90ec - Begin/end balance stat graph * b113a57b - Fix update function * 40da3bc2 - Listen on input * 903da795 - clean kfet statistic.js * 6d2e150a - clean align * 10d2b58f - clean some comments * ad2c8537 - Add Runpython (oops) * 5607d7c9 - Migration pour `kfet_forms` * b3418db3 - Merge branch 'Aufinal/edit_migration' into 'master' * f46ba0dd - Merge branch 'master' into Aufinal/inventaires * 32474a68 - Don't update input when unchecking * 51acb4e0 - Use new WS class * f13d1072 - Add simple tests for the stat views * df467767 - fix default GET param 'types' on operations stats * 7989a07b - cleaner fix * b383923d - Merge branch 'Kerl/test_stats' into aureplop/fix_stats * 0405eee7 - Merge branch 'Aufinal/inventaires' into 'master' * 278459e8 - typo....... * 72615bf4 - small fixes * 885e40fd - cleaner scales * dc07b072 - Flatten scale args in GET params of stats urls * ba11aa49 - categories are updatable * 73500069 - PEP8 * 85ba44c2 - Tests the redirection using the appropriate method * fe66a6ef - Merge branch 'aureplop/fix_stats' into 'master' * 42728e58 - Merge branch 'master' into qwann/k-fet/category_addcost * 3ee9de93 - few improvements on article category * ebf948d0 - Merge branch 'qwann/k-fet/category_addcost' into 'master' * a5671fdf - Merge branch 'master' into qwann/k-fet/kfet_open [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=376&start_sha=b0643c09288cdca6edd4c98c53d3884d13334fe1)
narmanli commented 2017-04-05 17:49:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * ef8fec89 - migration renamed [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=377&start_sha=a5671fdf4c15df6bd7ac950dc52865e496bbebac)
delobell commented 2017-04-05 18:03:07 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed target branch from k-fet to master

changed target branch from `k-fet` to `master`
delobell commented 2017-04-05 20:41:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Tu peux retirer ces trois lignes:

  • strict_ordering est par défaut à False
  • slight_ordering n'existe pas
Tu peux retirer ces trois lignes: - `strict_ordering` est par défaut à `False` - `slight_ordering` n'existe pas
delobell commented 2017-04-05 20:42:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

@narmanli du coup je reste sur l'avis que ce décorateur n'est pas nécessaire

@narmanli du coup je reste sur l'avis que ce décorateur n'est pas nécessaire
delobell commented 2017-04-06 20:04:53 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Tu peux aussi décorer dans l'URL pour faire permission_required('can_force_close')

Tu peux aussi décorer dans l'URL pour faire `permission_required('can_force_close')`
delobell commented 2017-04-06 20:12:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Parmi les nouveaux imports, seulement View semble être utilisé.

Parmi les nouveaux imports, seulement `View` semble être utilisé.
delobell commented 2017-04-06 20:15:00 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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 ?

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 ?
narmanli commented 2017-04-07 17:22:04 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 12 commits

  • ef8fec89...a0979c22 - 10 commits from branch master
  • 531a4611 - Merge branch 'master' into qwann/k-fet/kfet_open
  • deb0d4de - moving migration again

Compare with previous version

added 12 commits * ef8fec89...a0979c22 - 10 commits from branch `master` * 531a4611 - Merge branch 'master' into qwann/k-fet/kfet_open * deb0d4de - moving migration again [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=399&start_sha=ef8fec89fed8b401f0c8db714f6c498736395be2)
narmanli commented 2017-04-07 17:51:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * e18c2c69 - new migration [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=400&start_sha=deb0d4de1e4331dbc25ffb6dc9aef1934ceea58f)
lstephan commented 2017-04-07 18:03:59 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ils sont prévus pour rester ces logs ?

Ils sont prévus pour rester ces logs ?
lstephan commented 2017-04-07 18:04:15 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Idem

Idem
lstephan commented 2017-04-07 18:17:19 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Deux petites remarques esthétiques :

  • Les indications "Fermée" et "Ouverte" sont légèrement aggressives
  • Inversement, le widget d'ouverture est très discret ; si je n'avais pas vu son existence dans le code, je ne l'aurais presque pas remarqué.
Deux petites remarques esthétiques : - Les indications "Fermée" et "Ouverte" sont légèrement aggressives - Inversement, le widget d'ouverture est très discret ; si je n'avais pas vu son existence dans le code, je ne l'aurais presque pas remarqué.
narmanli commented 2017-04-07 23:32:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Oui :p

Oui :p
narmanli commented 2017-04-09 20:00:16 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 15873085 - small fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=413&start_sha=e18c2c698c4b38c9de7b98e12664570e42dcfb6e)
narmanli commented 2017-04-09 20:52:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * 5c6a73c5 - kfet_open uses moment.js [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=414&start_sha=15873085e11425d0cc1ca44ef3fd65c1091bfb8b)
narmanli commented 2017-04-10 16:45:35 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * cb9ba76a - small fixes [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=421&start_sha=5c6a73c59733dca6e0953b181ba00ab02500967e)
narmanli commented 2017-04-10 17:17:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * e0b0a531 - stupidness removed [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=422&start_sha=cb9ba76a4f7bde681c92f5f030e945d8b359d652)
lstephan commented 2017-04-10 17:40:41 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça donne un peu l'impression que cette fonction sert à fermer la K-Fêt, et pas à récupérer son état...

Ça donne un peu l'impression que cette fonction sert à fermer la K-Fêt, et pas à récupérer son état...
narmanli commented 2017-04-10 17:45:59 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * be8d249e - remove useless code [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/184/diffs?diff_id=424&start_sha=e0b0a531125f82378a7aa0182a70d5ccfc4a5b3b)
lstephan commented 2017-04-10 18:37:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Il y a cache.get_or_set qui fait exactement ce que tu fais ici.

Il y a `cache.get_or_set` qui fait exactement ce que tu fais ici.
delobell commented 2017-04-10 21:55:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !221

mentioned in merge request !221
delobell commented 2017-04-11 14:53:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link
  • En récupérant master, tu peux modifier le consumer KfetOpen pour le calquer sur celui KPsul (il va devenir très court :-P, et pas besoin d'user PermMixin à mon avis).
  • Il faudrait ranger KFET_OPEN et KFET_FORCE_CLOSE dans une classe.
  • Éventuellement utiliser kfet_config pour avoir une persistance. Rien de critique à ça cependant...
  • Éventuellement cacher le statut réel visible dans le JS si la personne n'a pas les permissions adéquates.

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 de force_close et KFetStatus.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 de raw_open et force_close, tandis que le premier WS est accessible par tous mais ne donne que le is_open.

- En récupérant master, tu peux modifier le consumer `KfetOpen` pour le calquer sur celui `KPsul` (il va devenir très court :-P, et pas besoin d'user PermMixin à mon avis). - Il faudrait ranger KFET_OPEN et KFET_FORCE_CLOSE dans une classe. - Éventuellement utiliser `kfet_config` pour avoir une persistance. Rien de critique à ça cependant... - Éventuellement cacher le statut réel visible dans le JS si la personne n'a pas les permissions adéquates. 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 de `force_close` et `KFetStatus.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 de `raw_open` et `force_close`, tandis que le premier WS est accessible par tous mais ne donne que le `is_open`.
mpepin commented 2017-05-14 17:42:05 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #65

mentioned in issue #65
narmanli commented 2017-05-15 19:24:49 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
narmanli commented 2017-05-15 19:57:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pas en 1.8 :(

Pas en 1.8 :(
narmanli commented 2017-05-15 19:58:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
mpepin commented 2017-05-15 21:04:05 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

typo: toutes

typo: toutes
mpepin commented 2017-05-15 21:07:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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 ?

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 ?
lstephan commented 2017-05-15 21:11:27 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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.

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.
mpepin commented 2017-05-15 21:16:33 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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 ?

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 ?
mpepin commented 2017-05-15 21:17:18 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je parle des /gestion au début des urls, pardon ce n'était pas très clair.

Je parle des `/gestion` au début des urls, pardon ce n'était pas très clair.
narmanli commented 2017-05-16 01:03:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Yup, j'ai vu, je suis en train de m'en occuper :)

Yup, j'ai vu, je suis en train de m'en occuper :)
narmanli commented 2017-05-16 01:04:46 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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 :)

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 :)
mpepin commented 2017-05-16 01:07:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ok mais si c'est du debug, c'est amené à rester en prod ?

Ok mais si c'est du debug, c'est amené à rester en prod ?
narmanli commented 2017-05-16 01:11:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

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 ^^

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 ^^
mpepin commented 2017-05-16 01:23:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Du coup laisse

Du coup laisse
delobell commented 2017-06-21 07:32:12 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !239

mentioned in merge request !239
mpepin commented 2017-06-23 00:29:16 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
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#488
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?