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

Donne les attributes à get_free_username #45

Merged
champeno merged 0 commits from Evarin/custom_usernames into master 2021-01-21 22:23:48 +01:00
Conversation 8 Commits - Files changed -
champeno commented 2021-01-17 21:36:53 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ajoute la possibilité d'overrider ENSCASBackend.get_free_username en utilisant les infos LDAP

Ajoute la possibilité d'overrider ENSCASBackend.get_free_username en utilisant les infos LDAP
lstephan commented 2021-01-18 12:10:45 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Si ça n'est pas forcément utile, je serais plus pour en faire un kwarg =)

Si ça n'est pas forcément utile, je serais plus pour en faire un `kwarg` =)
lstephan commented 2021-01-18 12:19:08 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça règlerait le problème des tests, en prime.

Ça règlerait le problème des tests, en prime.
champeno commented 2021-01-18 13:59:44 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

get_free_username(self, cas_login, attributes=None) ou get_free_username(self, cas_login, **kwargs) ?

L'intérêt d'être explicite sur attributes c'est qu'on sait quels sont les arguments qui sont donnés (ligne 105), puisqu'en héritant on contrôlera pas qui appelle avec quoi.

`get_free_username(self, cas_login, attributes=None)` ou `get_free_username(self, cas_login, **kwargs)` ? L'intérêt d'être explicite sur `attributes` c'est qu'on sait quels sont les arguments qui sont donnés (ligne 105), puisqu'en héritant on contrôlera pas qui appelle avec quoi.
lstephan commented 2021-01-18 14:00:35 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pensais plutôt à la première solution, oui !

Je pensais plutôt à la première solution, oui !
mpepin commented 2021-01-21 22:12:04 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>b8acee8d - Fix get_free_username tests</li></ul> [Compare with previous version](/klub-dev-ens/authens/-/merge_requests/23/diffs?diff_id=2328&start_sha=0b54b163503f74986f49d07f988ebf1bd8606fc5)
mpepin commented 2021-01-21 22:23:35 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

après discussion IRL en visio ça nous va bien finalement que ça reste comme ça

après discussion ~~IRL~~ en visio ça nous va bien finalement que ça reste comme ça
mpepin commented 2021-01-21 22:23:35 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all threads

resolved all threads
lstephan commented 2021-01-21 22:23:48 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit ef08eab248

mentioned in commit ef08eab248c7855f4294b4ed949deca22c84cba1
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
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#45
No description provided.
Delete branch "Evarin/custom_usernames"

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?