DGNum/gestioCOF
13
2
Fork 2
Code Issues 82 Pull requests 20 Projects Releases 12 Packages Wiki Activity Actions

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é

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