Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Catégorisation des actions pour les connexions directes #150

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ikarius
Copy link
Contributor

@ikarius ikarius commented Dec 23, 2024

Relatif à : https://trello.com/c/wWLEviB6/308-tableaux-de-suivi-migration-pro-connect

Ajoute un champ action dans le corps des logs permettra de contruire un
tableau de suivi plus facilement (les actions relatives à la connexion
directe sont catégorisées).

@ikarius ikarius self-assigned this Dec 23, 2024
@ikarius ikarius requested a review from ggounot December 23, 2024 10:56
back/dora/auth_links/views.py Dismissed Show dismissed Hide dismissed
Ajouter une action dans le corps du log permettra de contruire un
tableau de suivi plus facilement (les actions relatives à la connexion
directe sont catégorisées).
@ikarius ikarius force-pushed the auth-links-better-logging branch from 73a4864 to 32c8d30 Compare December 23, 2024 10:57
@ikarius ikarius enabled auto-merge (squash) December 23, 2024 11:00
@ikarius ikarius added the chore Technical improvement, refactoring, script, automation label Dec 23, 2024
Copy link
Contributor

@ggounot ggounot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀


class AuthLinkAction(enum.StrEnum):
"""
Actions loggées pour les liens d'identifications.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Actions loggées pour les liens d'identifications.
Actions loggées pour les liens d'identification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Technical improvement, refactoring, script, automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants