Skip to content
This repository was archived by the owner on Oct 24, 2023. It is now read-only.

Added import users action#123

Open
stevro wants to merge 3 commits intocode4romania:masterfrom
stevro:master
Open

Added import users action#123
stevro wants to merge 3 commits intocode4romania:masterfrom
stevro:master

Conversation

@stevro
Copy link

@stevro stevro commented Feb 29, 2020

  • added import users service which can be used to import users with any role
  • added csv parser which can be reused on other imports

Comment on lines +70 to +79
/*
* array(7) {
[0]=> "test popescu"
[1]=> "asfa@fas.asd"
[2]=> "Iasi"
[3]=> "Tg. frumos"
[4]=> "123123"
[5]=> "institutie slug"
[6]=> "organizatie slug"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

To make the code more easy to read you can replace this comment with some consts and use the consts in the following code:
const NAME = 0;
const EMAIL = 1;
const COUNTY = 2;
const CITY = 3;
const PHONE = 4;
const INSTITUTION = 5;
const ORGANIZATION = 6;

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants