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

[TECH] Migrer le script de création de comptes en masse (PIX-15276) #10520

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Nov 8, 2024

🍂 Problème

Le script de création en masse de compte n'utilise pas la nouvelle archi des scripts.

🌰 Proposition

Migrer le script sur la nouvelle architecture.

🪵 Pour tester

Lancer le script via la commande:

node src/identity-access-management/scripts/mass-create-user-accounts.js --help

Lancer le script avec un fichier CSV:

node src/identity-access-management/scripts/mass-create-user-accounts.js --file ./myfile.csv

Exemple de fichier CSV:

firstName,lastName,email,password
Dik,Tektive,[email protected],P@ssW0rd

L'utilisateur doit être créer en base de données avec une méthode de connexion (peut être aussi retrouvé via PixAdmin)

@bpetetot bpetetot self-assigned this Nov 8, 2024
Copy link

gitguardian bot commented Nov 8, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Test en local ✅

@bpetetot bpetetot changed the title [TECH] Migrate mass create use accounts script [TECH] Migrer le script de création de comptes en masse Nov 12, 2024
@bpetetot bpetetot changed the title [TECH] Migrer le script de création de comptes en masse [TECH] Migrer le script de création de comptes en masse (PIX-15276) Nov 12, 2024
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Lu et testé fonctionnellement avec succès

@pix-service-auto-merge pix-service-auto-merge merged commit 465ae56 into dev Nov 14, 2024
8 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech-migrate-script-20241108 branch November 14, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants