DGNum/gestioCOF

Fork 2

Utilitaires de dialogue #503

Merged

lstephan merged 0 commits from Aufinal/dialog_utils into aureplop/kpsul_js_refactor

2017-04-03 23:26:01 +02:00

lstephan commented

2017-03-26 23:02:39 +02:00

(Migrated from git.eleves.ens.fr)

Ajoute deux type de dialogue avec l'utilisateur :

une classe UserDialog pour ouvrir un simple dialogue jconfirm
une fonction api_with_auth pour gérer toutes les requêtes API pouvant nécessiter un mot de passe

Effet de bord : modifie légèrement le fonctionnement de la détection du capslock.

Ajoute deux type de dialogue avec l'utilisateur : - une classe `UserDialog` pour ouvrir un simple dialogue `jconfirm` - une fonction `api_with_auth` pour gérer toutes les requêtes API pouvant nécessiter un mot de passe Effet de bord : modifie légèrement le fonctionnement de la détection du capslock.

lstephan commented

2017-03-26 23:02:58 +02:00

(Migrated from git.eleves.ens.fr)

assigned to @delobell

lstephan commented

2017-03-26 23:10:36 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

08c752f1 - Simplify addcost management

Compare with previous version

added 1 commit * 08c752f1 - Simplify addcost management [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/199/diffs?diff_id=281&start_sha=5101400f64f606daf1837630206fbc4bae120f4d)

delobell commented

2017-03-31 21:42:43 +02:00

(Migrated from git.eleves.ens.fr)

Au lieu de errors, pre_content et pourquoi pas post_content.
Aussi vu ce que fait open et le nombre de choses qu'on pourrait lui faire faire. Je pense que c'est mieux d'avoir un seul tableau associatif en argument, avec aussi des valeurs par défaut.

Au lieu de `errors`, `pre_content` et pourquoi pas `post_content`. Aussi vu ce que fait `open` et le nombre de choses qu'on pourrait lui faire faire. Je pense que c'est mieux d'avoir un seul tableau associatif en argument, avec aussi des valeurs par défaut.

delobell commented

2017-03-31 21:42:52 +02:00

(Migrated from git.eleves.ens.fr)

C'est du pinaillage mais juste defaults comme nom pour des attributs quelconques, j'trouve que ça passe mieux

C'est du pinaillage mais juste `defaults` comme nom pour des attributs quelconques, j'trouve que ça passe mieux

delobell commented

2017-03-31 21:47:07 +02:00

(Migrated from git.eleves.ens.fr)

Quand est-ce que sert cette distinction ?
Juste le premier cas semble plus sympa

Quand est-ce que sert cette distinction ? Juste le premier cas semble plus sympa

lstephan commented

2017-03-31 21:51:18 +02:00

(Migrated from git.eleves.ens.fr)

Le 2e cas me permet de pas avoir de dictionnaire quand on a qu'un seul argument, donc c'est le comportement par défaut ; pour l'instant j'utilise un dict uniquement pour addCost.

Le 2e cas me permet de pas avoir de dictionnaire quand on a qu'un seul argument, donc c'est le comportement par défaut ; pour l'instant j'utilise un `dict` uniquement pour `addCost`.

delobell commented

2017-03-31 22:33:42 +02:00

(Migrated from git.eleves.ens.fr)

Ok effectivement, pour le cas où il n'y a qu'un input c'est sympa. Mais du coup tu peux prendre le code du premier block, puis checker la taille du dict ?

delobell commented

2017-03-31 22:40:25 +02:00

(Migrated from git.eleves.ens.fr)

Les deux window.lock = 0 peuvent se mettre dans un .always(function() { window.lock = 0; })

Les deux `window.lock = 0` peuvent se mettre dans un `.always(function() { window.lock = 0; })`

lstephan commented

2017-03-31 23:10:17 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

582cdeba - Better callback management
236dcb46 - Tweaks to UserDialog

Compare with previous version

added 2 commits * 582cdeba - Better callback management * 236dcb46 - Tweaks to UserDialog [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/199/diffs?diff_id=310&start_sha=08c752f1b399930144e97ce2fcce92c8294b7706)

delobell commented

2017-03-31 23:59:46 +02:00

(Migrated from git.eleves.ens.fr)

resolved all discussions

delobell commented

2017-04-01 12:24:06 +02:00

(Migrated from git.eleves.ens.fr)

Y sert plus celui-là

delobell commented

2017-04-01 12:27:11 +02:00

(Migrated from git.eleves.ens.fr)

Pour clarifier, tu peux appeler le premier data qqch du type inputs ou inputs_values ou whatever
Et garder data pour le callback du on_400.

Pour clarifier, tu peux appeler le premier `data` qqch du type `inputs` ou `inputs_values` ou whatever Et garder `data` pour le callback du `on_400`.

lstephan commented

2017-04-01 14:21:53 +02:00

(Migrated from git.eleves.ens.fr)

Le dialogue d'annulation apparaît chez moi...

lstephan commented

2017-04-01 14:34:12 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

29836fd1 - Remove deprecated option
4af25621 - More clarity in argument names

Compare with previous version

added 2 commits * 29836fd1 - Remove deprecated option * 4af25621 - More clarity in argument names [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/199/diffs?diff_id=320&start_sha=236dcb4644ac1b619f85f9b21c73889e88cc456b)

delobell commented

2017-04-03 14:22:39 +02:00

(Migrated from git.eleves.ens.fr)

Si window.lock n'est plus accédé ailleurs que dans api_with_auth, on peut le virer d'ici (ça fait un truc random de moins qui traîne). Le premier appel de cette fonction va quand même fonctionner même sans créer cette variable avant.
Par ailleurs pour pas s’emmêler les pinceaux, on pourrait la renommer en api_lock dans cette fonction.

Si `window.lock` n'est plus accédé ailleurs que dans `api_with_auth`, on peut le virer d'ici (ça fait un truc random de moins qui traîne). Le premier appel de cette fonction va quand même fonctionner même sans créer cette variable avant. Par ailleurs pour pas s’emmêler les pinceaux, on pourrait la renommer en `api_lock` dans cette fonction.

lstephan commented

2017-04-03 21:08:49 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

50200371 - api_lock inside kfet.js

Compare with previous version

added 1 commit * 50200371 - api_lock inside kfet.js [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/199/diffs?diff_id=340&start_sha=4af2562121c9e032804518ec51237dcc26cbd88b)

delobell commented

2017-04-03 23:22:42 +02:00

(Migrated from git.eleves.ens.fr)

resolved all discussions

delobell commented

2017-04-03 23:25:57 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit dcda67aaf7

mentioned in commit dcda67aaf71cd41bc1d104f0097c581d8549e25d

delobell commented

2017-04-03 23:26:01 +02:00

(Migrated from git.eleves.ens.fr)