entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 32 Releases Activity

authenticators: add helper functions for show condition (#67986) #285

Merged
ecazenave merged 1 commits from wip/67986-auth-conditions into main 2024-03-29 09:11:56 +01:00
Conversation 1 Commits 1 Files Changed 3 +100 -1
ecazenave commented 2024-03-21 18:07:22 +01:00
Owner
Copy Link
No description provided.
ecazenave added 1 commit 2024-03-21 18:07:23 +01:00
gitea/authentic/pipeline/head There was a failure building this commit Details
696d274015 authenticators: add helper functions for show condition (#67986)
ecazenave force-pushed wip/67986-auth-conditions from 696d274015 to 8a65a2b33c 2024-03-22 10:49:18 +01:00 Compare
ecazenave changed title from WIP: authenticators: add helper functions for show condition (#67986) to authenticators: add helper functions for show condition (#67986) 2024-03-22 11:03:30 +01:00
ecazenave commented 2024-03-22 11:06:44 +01:00
Author
Owner
Copy Link

J'avais commencé le travail sur l'interface mais je me suis embourbé jusqu'à laisser tomber.

En me disant que l'essentiel est ailleurs à savoir de nous mâcher le travail, d'avoir la même chose partout et de pouvoir personnaliser le comportement à notre sauce.

Il manque une migration pour actualiser le help_text du champ show_condition, je ferai ça quand le gros ticket sur le squash des migrations sera passé.

J'avais commencé le travail sur l'interface mais je me suis embourbé jusqu'à laisser tomber. En me disant que l'essentiel est ailleurs à savoir de nous mâcher le travail, d'avoir la même chose partout et de pouvoir personnaliser le comportement à notre sauce. Il manque une migration pour actualiser le help_text du champ show_condition, je ferai ça quand le gros ticket sur le squash des migrations sera passé.
ecazenave force-pushed wip/67986-auth-conditions from 8a65a2b33c to 36bc55d3ce 2024-03-22 11:08:42 +01:00 Compare
pmarillonnet approved these changes 2024-03-28 14:45:27 +01:00
pmarillonnet left a comment
Owner
Copy Link

Ok, c’est déjà une bonne première étape, et ça permet de dégager de l’UI cette notion de login_hint qu’un admin fonctionnel paramétrant le moyen d’authentification n’a pas à connaître. Ack.

Ok, c’est déjà une bonne première étape, et ça permet de dégager de l’UI cette notion de `login_hint` qu’un admin fonctionnel paramétrant le moyen d’authentification n’a pas à connaître. Ack.
src/authentic2/apps/authenticators/models.py
@ -138,0 +139,4 @@
def is_for_office(self, office_keyword, ctx):
try:
return evaluate_condition(
settings.AUTHENTICATOR_SHOW_CONDITIONS[office_keyword], ctx, on_raise=False
pmarillonnet commented 2024-03-28 14:35:13 +01:00
Owner
Copy Link

on_raise=False pour ce bout de condition d’affichage, ok, très bien.

`on_raise=False` pour ce bout de condition d’affichage, ok, très bien.
ecazenave merged commit 36bc55d3ce into main 2024-03-29 09:11:56 +01:00
ecazenave deleted branch wip/67986-auth-conditions 2024-03-29 09:11:56 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
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: entrouvert/authentic#285
No description provided.
Delete Branch "wip/67986-auth-conditions"

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?