PGL/Clyde
PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

Register API and form optimization [depend: backend /register] #93

Merged
tonitch merged 2 commits from tonitch/front/register into master 2024-03-15 14:46:40 +01:00
Conversation 0 Commits 2 Files Changed 2 +19 -23
1 changed files with 19 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e6dd0a8995 - Show all commits

42
frontend/src/rest/Users.js
View File

@ -7,18 +7,17 @@ export async function login(user, pass, exp){
/**
* Register a user (tokenless)
*
* @param data object containing info about the user
*
* - firstname
* - lastname
* - birthdate
* - password
* - mail
* - address
* - country
* - cursus
* @param firstname
* @param lastname
* @param birthdate
* @param password
* @param mail
* @param address
* @param country
* @param cursus
* @param imageId id of the image in database returned when uploaded
*/
export async function register(firstname, lastname, birthDate, password, email, address, country, cursus){
export async function register(firstname, lastname, birthDate, password, email, address, country, cursus, imageId){
return restPost("/register", {
Maxime marked this conversation as resolved
Maxime commented 2024-03-12 23:21:23 +01:00
Review
Copy Link

ProfilePictureUrl pas présent ?

ProfilePictureUrl pas présent ?
Maxime commented 2024-03-12 23:25:18 +01:00
Review
Copy Link

car de mémoire tu postais l'image directe je te retournais l'URL comme ça tu la postais

car de mémoire tu postais l'image directe je te retournais l'URL comme ça tu la postais
tonitch commented 2024-03-13 10:40:39 +01:00
Review
Copy Link

c'est parce que j'avais fait ça avant qu'on parle des pp 👉👈

c'est parce que j'avais fait ça avant qu'on parle des pp 👉👈
tonitch commented 2024-03-13 13:09:35 +01:00
Review
Copy Link

@Maxime Le problème c'est que si tu me retourne l'url tu vas pas pouvoir le lier à la db
faudrait du coup me retourner l'id que je te retourne. Ce qui n'est pas ouf non plus mais bon. en vrai autant ne pas trop se casser la tête

@Maxime Le problème c'est que si tu me retourne l'url tu vas pas pouvoir le lier à la db faudrait du coup me retourner l'id que je te retourne. Ce qui n'est pas ouf non plus mais bon. en vrai autant ne pas trop se casser la tête
firstname: firstname,
lastname: lastname,
@ -34,21 +33,19 @@ export async function register(firstname, lastname, birthDate, password, email,
/**
* Register a user (by secretary)
*
* @param data object containing info about the user
*
* - firstname
* - lastname
* - birthdate
* - password
* - mail
* - address
* - country
* - cursus
* @param firstname
* @param lastname
* @param birthdate
* @param password
* @param mail
* @param address
* @param country
* @param imageId id of the image in database returned when uploaded
*
* PS: the password is not is not required as it is generated by the backend and sent to the user
* by mail. it's up to the user to change it if he cares about security
*/
export async function createUser(firstname, lastname, birthDate, email, address, country, cursus, role){
export async function createUser(firstname, lastname, birthDate, email, address, country, role, imageId){
return restPost("/user", {
firstname: firstname,
lastname: lastname,
Maxime commented 2024-03-12 23:19:15 +01:00
Review
Copy Link

pas necessairement de cursus, surtout dans le cadre où le secrétariat postent les teachers

(surtout que les users on pas de cursus)

pas necessairement de cursus, surtout dans le cadre où le secrétariat postent les teachers (surtout que les users on pas de cursus)
👍 1
@ -57,7 +54,6 @@ export async function createUser(firstname, lastname, birthDate, email, address,
email: email,
address: address,
country: country,
cursus:cursus
});
}