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

Update database.csv #210

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

Conversation

Boukrim212
Copy link

Nous essayons de limiter au maximum l'ajout d'identités dans le fichier database.csv pour réduire les coûts de maintenance et limiter les risques de non conformité au RGPD.

Si vous avez modifié ce fichier, pouvez vous nous préciser ce qui vous a manqué dans les identités déjà présentes dans le fichier et en quoi vos identités résolvent le problème ?

@Boukrim212
Copy link
Author

Please add this commit, I need to test with this user

@benratti
Copy link
Contributor

Bonjour,

Le fichier database.csv est fait pour avoir un jeu de données commun à tous les partenaires de FranceConnect qui souhaitent réaliser des tests sur notre environnement d'intégration.

Nous vous remercions d'utiliser dans ce fichier des données qui peuvent être utile à l'ensemble des partenaires.
Pouvez vous préciser l'utilité d'ajouter cette identité dans le jeu de donnée ?

S'il n'y a pas d'utilisé commune, nous vous invitons à réutiliser une identité déjà présente ( les cas représentations sont normalement déjà présent ). Si vous souhaitez un jeu de données spécifique à vos fs, nous vous invitons à utiliser le répertoire /data/ qui est destiné à recevoir des fichiers spéciques par partenaire.

Copy link
Contributor

@benratti benratti left a comment

Choose a reason for hiding this comment

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

Nécessite d'apporter des précisions sur l'interet d'ajouter cette nouvelle identité (cf mon commentaire de la pull-request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants