DGNum/authens
17
0
Fork 0
Code Issues 2 Pull requests Projects Releases 2 Packages Wiki Activity

Backend pour authentification des vieilleux #33

Merged
lstephan merged 0 commits from Aufinal/old_cas into master 2020-06-13 18:19:01 +02:00
Conversation 19 Commits - Files changed -
lstephan commented 2020-06-12 20:43:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Les vieilleux peuvent s'authentifier via un backend spécifique, avec leur ancien login clipper, leur promo et un mot de passe.

TODO (pour une autre MR):

  • vue d'auth proprement dite
  • mécanisme de deprecation : automatique ou non ?
  • première création d'un mot de passe : possible de le faire avant la fin d'un clipper ?
Les vieilleux peuvent s'authentifier via un backend spécifique, avec leur ancien login clipper, leur promo et un mot de passe. TODO (pour une autre MR): - vue d'auth proprement dite - mécanisme de deprecation : automatique ou non ? - première création d'un mot de passe : possible de le faire avant la fin d'un clipper ?
lstephan commented 2020-06-12 20:43:36 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed milestone to %1

changed milestone to %1
lstephan commented 2020-06-12 20:46:08 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

assigned to @mpepin

assigned to @mpepin
lstephan commented 2020-06-12 23:51:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Remarque : cette contrainte est spécifique à Django>=2.2; est-ce qu'on veut garder ça ou est-ce qu'on garde un unique_together (sachant que c'est deprecated soon) ?

Remarque : [cette contrainte](https://git.eleves.ens.fr/klub-dev-ens/authens/merge_requests/11/diffs#13bd4d2eec027826fd86400e779fd82018d8d17f_40_67) est spécifique à Django>=2.2; est-ce qu'on veut garder ça ou est-ce qu'on garde un `unique_together` (sachant que c'est deprecated soon) ?
mpepin commented 2020-06-13 17:18:52 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je ne connaissais pas cette nouvelle façon de faire. Ça me va de garder ça : la version 2.2 est la plus ancienne supportée donc ça me paraît legit d'utiliser des features ≥2.2-only

Je ne connaissais pas cette nouvelle façon de faire. Ça me va de garder ça : la version 2.2 est la plus ancienne supportée donc ça me paraît legit d'utiliser des features ≥2.2-only
mpepin commented 2020-06-13 17:25:44 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je mettrais quelque chose de plus explicite ici, par exemple :

If a CAS account exists, but with the wrong entrance year, turn it (ou "deprecate" si tu y tiens mais je ne suis pas sûr que "deprecate" soit clair) into an OldCASAccount and create a new (ou "fresh" ?) CASAccount.

Je mettrais quelque chose de plus explicite ici, par exemple : If a CAS account exists, but with the wrong entrance year, turn it (ou "deprecate" si tu y tiens mais je ne suis pas sûr que "deprecate" soit clair) into an OldCASAccount and create a new (ou "fresh" ?) CASAccount.
lstephan commented 2020-06-13 17:34:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](/klub-dev-ens/authens/merge_requests/11/diffs?diff_id=1778&start_sha=bb3aac63b9b7f0ae6393cada2901b23969c94446#f1528d07ad4f24510aa58c36fb7708cb5663435e_105_104)
lstephan commented 2020-06-13 17:34:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • c8c24176 - Better doc

Compare with previous version

added 1 commit <ul><li>c8c24176 - Better doc</li></ul> [Compare with previous version](/klub-dev-ens/authens/merge_requests/11/diffs?diff_id=1778&start_sha=bb3aac63b9b7f0ae6393cada2901b23969c94446)
mpepin commented 2020-06-13 17:37:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

pur chipotage : je dirais "mitigate" au lieu de "avoid"

pur chipotage : je dirais "mitigate" au lieu de "avoid"
mpepin commented 2020-06-13 17:38:08 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Comme dit plus haut ça me va très bien

Comme dit plus haut ça me va très bien
mpepin commented 2020-06-13 17:39:26 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Note pour plus tard : il faudra expliquer comment configurer ça dans le README quand on aura terminé

Note pour plus tard : il faudra expliquer comment configurer ça dans le README quand on aura terminé
mpepin commented 2020-06-13 17:42:58 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

created #11 to continue this discussion

created #11 to continue this discussion
lstephan commented 2020-06-13 18:04:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](/klub-dev-ens/authens/merge_requests/11/diffs?diff_id=1779&start_sha=c8c24176ed1863f0062e079b1acd3711cc6b3fc2#f1528d07ad4f24510aa58c36fb7708cb5663435e_154_161)
lstephan commented 2020-06-13 18:04:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>0d4f848f - Better doc</li></ul> [Compare with previous version](/klub-dev-ens/authens/merge_requests/11/diffs?diff_id=1779&start_sha=c8c24176ed1863f0062e079b1acd3711cc6b3fc2)
mpepin commented 2020-06-13 18:18:14 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je me permets de blackifier la migration et je merge, c'est hyper propre !

Je me permets de blackifier la migration et je merge, c'est hyper propre !
mpepin commented 2020-06-13 18:18:34 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>5047ee14 - Black</li></ul> [Compare with previous version](/klub-dev-ens/authens/merge_requests/11/diffs?diff_id=1781&start_sha=0d4f848fe59c48deffdbe74b4c813c6b0d7affcc)
mpepin commented 2020-06-13 18:18:50 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
mpepin commented 2020-06-13 18:19:01 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
mpepin commented 2020-06-13 18:19:01 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit e01ddbbcf5

mentioned in commit e01ddbbcf5eee84f9d3e141425d597e7c60f4dae
mpepin commented 2020-06-13 18:23:07 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #9

mentioned in issue #9
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
documentation

   
Doing

   
Doing

   
Good first issue

   
To Do

   
To Do
  
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
documentation
Doing
Doing
Good first issue
To Do
To Do
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
v1
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/authens#33
No description provided.
Delete branch "Aufinal/old_cas"

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?