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

Fonctionnement du négatif + erreurs de K-Psul #798

Merged
lstephan merged 0 commits from Aufinal/remove_negative into master 2021-06-17 21:22:15 +02:00
Conversation 8 Commits - Files changed -
lstephan commented 2021-02-28 02:13:41 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

On rework le négatif tel que décrit dans #279:

  • plus possible de passer en dessous de kfet_config.overdraft_amount de négatif
  • plus de limite de temps sur les négatifs
  • plus d'autorisation de négatif.

On en profite pour mettre un système anti-petits malins : il n'est plus possible de reset la durée de son négatif en faisant puis annulant une charge (utile pour !492 ). Cela peut être amélioré en enlevant la permission cancel_old_operations aux wo.men.

Accessoirement, la suppression de vieilles opérations est réparée (elle était cassée depuis 10 mois).

On rework aussi complètement l'affichage des erreurs sur K-Psul: les erreurs sont générées par le serveur, et sont donc plus explicites. Pour permettre des tests moins spécifiques, on rajoute des "codes" aux erreurs (comme pour les ValidationErrors).

Fixes #279

Dépend de !493

On rework le négatif tel que décrit dans #279: - plus possible de passer en dessous de `kfet_config.overdraft_amount` de négatif - plus de limite de temps sur les négatifs - plus d'autorisation de négatif. On en profite pour mettre un système anti-petits malins : il n'est plus possible de reset la durée de son négatif en faisant puis annulant une charge (utile pour !492 ). Cela peut être amélioré en enlevant la permission `cancel_old_operations` aux wo.men. Accessoirement, la suppression de vieilles opérations est réparée (elle était cassée depuis 10 mois). On rework aussi complètement l'affichage des erreurs sur K-Psul: les erreurs sont générées par le serveur, et sont donc plus explicites. Pour permettre des tests moins spécifiques, on rajoute des "codes" aux erreurs (comme pour les `ValidationError`s). Fixes #279 Dépend de !493
lstephan commented 2021-02-28 02:13:54 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
lstephan commented 2021-02-28 02:16:47 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 30cd7d9a - Oublis de renaming

Compare with previous version

added 1 commit <ul><li>30cd7d9a - Oublis de renaming</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/494/diffs?diff_id=2458&start_sha=6d389c2390143390b20f6edb65c3d6677375a59f)
lstephan commented 2021-06-15 14:11:59 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 47 commits

  • 30cd7d9a...0351f672 - 32 commits from branch master
  • 032ae3ce - Change les permissions pour geler/dégeler un compte
  • ba4844a5 - Unfreeze every account
  • 35d6bb24 - New frozen function
  • 8c386c51 - Frozen error display
  • e131b068 - Remove unused permission
  • 7c8a9c55 - Fix decorator
  • 5c411175 - Remove print
  • bd802920 - Fix test
  • ad43b4bc - Nouveau fonctionnement des négatifs
  • 9d08fc07 - Migration
  • 25430e40 - Tests du nouveau comportement
  • 4b424ebe - Nouvelle gestion des erreurs JSON
  • cbf09139 - Adapte le JS aux nouvelles erreurs
  • b9003fdf - Tests
  • c4faba83 - Oublis de renaming

Compare with previous version

added 47 commits <ul><li>30cd7d9a...0351f672 - 32 commits from branch <code>master</code></li><li>032ae3ce - Change les permissions pour geler/dégeler un compte</li><li>ba4844a5 - Unfreeze every account</li><li>35d6bb24 - New frozen function</li><li>8c386c51 - Frozen error display</li><li>e131b068 - Remove unused permission</li><li>7c8a9c55 - Fix decorator</li><li>5c411175 - Remove print</li><li>bd802920 - Fix test</li><li>ad43b4bc - Nouveau fonctionnement des négatifs</li><li>9d08fc07 - Migration</li><li>25430e40 - Tests du nouveau comportement</li><li>4b424ebe - Nouvelle gestion des erreurs JSON</li><li>cbf09139 - Adapte le JS aux nouvelles erreurs</li><li>b9003fdf - Tests</li><li>c4faba83 - Oublis de renaming</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/494/diffs?diff_id=2968&start_sha=30cd7d9aa75e313a09fe14819a81a136b5611f81)
lstephan commented 2021-06-17 10:42:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 20 commits

  • c4faba83...87433011 - 13 commits from branch master
  • ef8c1b8b - Nouveau fonctionnement des négatifs
  • 348881d2 - Migration
  • 1939a54f - Tests du nouveau comportement
  • 29236e0b - Nouvelle gestion des erreurs JSON
  • 964eec6a - Adapte le JS aux nouvelles erreurs
  • 4205e0ad - Tests
  • 4326ba90 - Oublis de renaming

Compare with previous version

added 20 commits <ul><li>c4faba83...87433011 - 13 commits from branch <code>master</code></li><li>ef8c1b8b - Nouveau fonctionnement des négatifs</li><li>348881d2 - Migration</li><li>1939a54f - Tests du nouveau comportement</li><li>29236e0b - Nouvelle gestion des erreurs JSON</li><li>964eec6a - Adapte le JS aux nouvelles erreurs</li><li>4205e0ad - Tests</li><li>4326ba90 - Oublis de renaming</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/494/diffs?diff_id=2985&start_sha=c4faba835afb996f46e9da622af0fdc9a0923da7)
lstephan commented 2021-06-17 10:49:40 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>4060730e - Remove logging</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/494/diffs?diff_id=2988&start_sha=4326ba901612550276fbf0c75a7524530da5fbd1)
thubrecht commented 2021-06-17 17:09:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Si un compte est en négatif, son profil plante, dans le template kfet/left_account.html y'a des références à authz_overdraft_until

Si un compte est en négatif, son profil plante, dans le template `kfet/left_account.html` y'a des références à `authz_overdraft_until`
lstephan commented 2021-06-17 17:22:21 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>6b316c48 - Remove obsolete section</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/494/diffs?diff_id=2990&start_sha=4060730ec53bd129403c886b4e404cdb3780f6ca)
thubrecht commented 2021-06-17 21:22:15 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit a5c822e7f7

mentioned in commit a5c822e7f7ef159119883d97fa0e64a5a5af212b
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é

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
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#798
No description provided.
Delete branch "Aufinal/remove_negative"

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?