Ajoute les transferts dans l'historique #436
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#436
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "Aufinal/transferts_historique"
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?
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>
etk-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
ettransfers.html
se met maintenant à jour automatiquement via WS.Fix #77
Added 2 commits:
0b61a48c
- fix selection2c2da60e
- send data for cancelAdded 1 commit:
49bef61e
- filter transfers frop opesAdded 2 commits:
66304359
- unite cancel_ope and cancel_transfer1dbbad38
- transfer cancellation htmlmentioned in issue #115
Added 4 commits:
3f35dc2c
- unite transfer historya9e1cd01
- add transfersonly optionee54b366
- minor imprevements to historye52c4458
- pluralize functionUnmarked this merge request as a Work In Progress
Added 1 commit:
f06a732d
- remove unnecessary functionAdded 92 commits:
k-fet
8895daff
- Merge branch 'Aufinal/transferts_historique' of git.eleves.ens.fr:cof-geek/gesti…f8aa6772
- fix socket updateAdded 1 commit:
1ea33434
- ws update for history and transfer pagesAdded 2 commits:
1d5e6930
- ws for transfersb404c989
- pep8Added 1 commit:
db94a890
- remove addition of new opegroupsÇ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)
Idem
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.
Voire même
Math.abs(count)
si count peut être négatif.(#chipotage)
mentioned in issue #144
Plutôt que d'avoir 3 pluralize :
ou qqch du genre
Indeed, ça peut permettre de l'étendre plus encore si nécessaire.
(T'entends quoi par 'utiliser de l'objet' ?)
Added 3 commits:
fc2de20a
- use switch + move text02735642
- better pluralize48140925
- simpler string pluralizingDe la POO (/Prototype). Le code en sera plus clair, plus facilement manipulable, maintenable, des interfaces claires, etc
Je suis en train de faire ça. Je vais déjà cleaner les bases (article, caisse...) pour ensuite pouvoir construire du schmilblick au-dessus.
Resolved all discussions
Status changed to closed
Status changed to reopened
Status changed to closed
mentioned in merge request !183
mentioned in merge request !192
mentioned in issue #77
Pull request closed