Ajoute les transferts dans l'historique #436

Closed
lstephan wants to merge 0 commits from Aufinal/transferts_historique into k-fet
lstephan commented 2016-12-09 20:37:12 +01:00 (Migrated from git.eleves.ens.fr)

Fusionne totalement l'affichage des transferts dans l'historique avec celui des opérations normales. Les transferts sont maintenant affichés sur k-fet/k-psul, k-fet/accounts/<trigramme> et k-fet/history.

La page k-fet/transfers utilise maintenant le même js que les autres, et les fonctions d'annulation d'opérations ont été fusionnées.

Le filtrage par caisse dans k-fet/history n'affiche pas les transferts.

L'historique de history.html et transfers.html se met maintenant à jour automatiquement via WS.

Fix #77

Fusionne totalement l'affichage des transferts dans l'historique avec celui des opérations normales. Les transferts sont maintenant affichés sur `k-fet/k-psul`, `k-fet/accounts/<trigramme>` et `k-fet/history`. La page `k-fet/transfers` utilise maintenant le même js que les autres, et les fonctions d'annulation d'opérations ont été fusionnées. Le filtrage par caisse dans `k-fet/history` n'affiche pas les transferts. L'historique de `history.html` et `transfers.html` se met maintenant à jour automatiquement via WS. Fix #77
lstephan commented 2016-12-11 02:52:40 +01:00 (Migrated from git.eleves.ens.fr)

Added 2 commits:

Added 2 commits: * 0b61a48c - fix selection * 2c2da60e - send data for cancel
lstephan commented 2016-12-11 17:47:25 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

Added 1 commit: * 49bef61e - filter transfers frop opes
lstephan commented 2016-12-12 00:00:53 +01:00 (Migrated from git.eleves.ens.fr)

Added 2 commits:

  • 66304359 - unite cancel_ope and cancel_transfer
  • 1dbbad38 - transfer cancellation html
Added 2 commits: * 66304359 - unite cancel_ope and cancel_transfer * 1dbbad38 - transfer cancellation html
lstephan commented 2016-12-12 02:15:28 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in issue #115

mentioned in issue #115
lstephan commented 2016-12-12 02:23:32 +01:00 (Migrated from git.eleves.ens.fr)

Added 4 commits:

Added 4 commits: * 3f35dc2c - unite transfer history * a9e1cd01 - add transfersonly option * ee54b366 - minor imprevements to history * e52c4458 - pluralize function
lstephan commented 2016-12-12 02:23:50 +01:00 (Migrated from git.eleves.ens.fr)

Unmarked this merge request as a Work In Progress

Unmarked this merge request as a Work In Progress
lstephan commented 2016-12-12 02:29:15 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

  • f06a732d - remove unnecessary function
Added 1 commit: * f06a732d - remove unnecessary function
lstephan commented 2017-02-05 05:21:10 +01:00 (Migrated from git.eleves.ens.fr)

Added 92 commits:

  • f06a732d...5b0dc1a0 - 90 commits from branch k-fet
  • 8895daff - Merge branch 'Aufinal/transferts_historique' of git.eleves.ens.fr:cof-geek/gesti…
  • f8aa6772 - fix socket update
Added 92 commits: * f06a732d...5b0dc1a0 - 90 commits from branch `k-fet` * 8895daff - Merge branch &#x27;Aufinal&#x2F;transferts_historique&#x27; of git.eleves.ens.fr:cof-geek&#x2F;gesti… * f8aa6772 - fix socket update
lstephan commented 2017-02-05 06:05:20 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

  • 1ea33434 - ws update for history and transfer pages
Added 1 commit: * 1ea33434 - ws update for history and transfer pages
lstephan commented 2017-02-05 06:42:34 +01:00 (Migrated from git.eleves.ens.fr)

Added 2 commits:

Added 2 commits: * 1d5e6930 - ws for transfers * b404c989 - pep8
lstephan commented 2017-02-07 21:46:13 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

  • db94a890 - remove addition of new opegroups
Added 1 commit: * db94a890 - remove addition of new opegroups
delobell commented 2017-02-11 23:08:52 +01:00 (Migrated from git.eleves.ens.fr)

Ça me semble être mieux pour la clarté de faire qqch comme : if (opegroup['type'] == 'opegroup') { ... else if (opegroup['type'] == 'transfergroup') { ... }
Ça pourrait même être un switch-case.
(Et pour que ça soit au moins écrit une fois: utiliser de l'objet semble plus que legit dans le JS K-Psul/Fêt)

Ça me semble être mieux pour la clarté de faire qqch comme : ` if (opegroup['type'] == 'opegroup') { ... else if (opegroup['type'] == 'transfergroup') { ... }` Ça pourrait même être un switch-case. (Et pour que ça soit au moins écrit une fois: utiliser de l'objet semble plus que legit dans le JS K-Psul/Fêt)
delobell commented 2017-02-11 23:13:19 +01:00 (Migrated from git.eleves.ens.fr)

Idem

Idem
delobell commented 2017-02-11 23:23:34 +01:00 (Migrated from git.eleves.ens.fr)

Ceci fonctionne grâce au fait qu'il n'y a pas de tag html avec une class info dans le template d'un opegroup.
Ça ne devrait pas être là. C'est un cas spécifique aux transferts et on est dans un contexte général.

Ceci fonctionne grâce au fait qu'il n'y a pas de tag html avec une class info dans le template d'un opegroup. Ça ne devrait pas être là. C'est un cas spécifique aux transferts et on est dans un contexte général.
delobell commented 2017-02-11 23:38:52 +01:00 (Migrated from git.eleves.ens.fr)
if (count >= 2)
    return irreg_plural ? irreg_plural : this + 's';
return this;

Voire même Math.abs(count) si count peut être négatif.
(#chipotage)

```javascript if (count >= 2) return irreg_plural ? irreg_plural : this + 's'; return this; ``` Voire même `Math.abs(count)` si count peut être négatif. (#chipotage)
manet commented 2017-02-11 23:53:14 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in issue #144

mentioned in issue #144
delobell commented 2017-02-11 23:54:47 +01:00 (Migrated from git.eleves.ens.fr)

Plutôt que d'avoir 3 pluralize :

var content = nb+' opération va être annulée'.pluralize(nb, ' opérations vont être annulées')

ou qqch du genre

Plutôt que d'avoir 3 pluralize : ```javascript var content = nb+' opération va être annulée'.pluralize(nb, ' opérations vont être annulées') ``` ou qqch du genre
lstephan commented 2017-02-12 00:16:17 +01:00 (Migrated from git.eleves.ens.fr)

Indeed, ça peut permettre de l'étendre plus encore si nécessaire.

(T'entends quoi par 'utiliser de l'objet' ?)

Indeed, ça peut permettre de l'étendre plus encore si nécessaire. (T'entends quoi par 'utiliser de l'objet' ?)
lstephan commented 2017-02-12 00:44:55 +01:00 (Migrated from git.eleves.ens.fr)

Added 3 commits:

Added 3 commits: * fc2de20a - use switch + move text * 02735642 - better pluralize * 48140925 - simpler string pluralizing
delobell commented 2017-02-12 06:22:59 +01:00 (Migrated from git.eleves.ens.fr)

De la POO (/Prototype). Le code en sera plus clair, plus facilement manipulable, maintenable, des interfaces claires, etc

De la POO (/Prototype). Le code en sera plus clair, plus facilement manipulable, maintenable, des interfaces claires, etc
delobell commented 2017-02-12 06:24:22 +01:00 (Migrated from git.eleves.ens.fr)

Je suis en train de faire ça. Je vais déjà cleaner les bases (article, caisse...) pour ensuite pouvoir construire du schmilblick au-dessus.

Je suis en train de faire ça. Je vais déjà cleaner les bases (article, caisse...) pour ensuite pouvoir construire du schmilblick au-dessus.
delobell commented 2017-02-13 16:50:41 +01:00 (Migrated from git.eleves.ens.fr)

Resolved all discussions

Resolved all discussions
lstephan commented 2017-02-23 12:48:38 +01:00 (Migrated from git.eleves.ens.fr)

Status changed to closed

Status changed to closed
lstephan commented 2017-02-23 12:49:01 +01:00 (Migrated from git.eleves.ens.fr)

Status changed to reopened

Status changed to reopened
lstephan commented 2017-03-16 01:21:39 +01:00 (Migrated from git.eleves.ens.fr)

Status changed to closed

Status changed to closed
lstephan commented 2017-03-19 00:10:48 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in merge request !183

mentioned in merge request !183
lstephan commented 2017-03-19 04:39:58 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in merge request !192

mentioned in merge request !192
lstephan commented 2019-12-18 19:46:55 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in issue #77

mentioned in issue #77

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
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#436
No description provided.