Update CharacterController with Either, also small style fix in UserController #30

Merged
s426134 merged 1 commits from SES-92 into dev 2020-12-16 21:49:36 +01:00
Owner

Natalia dała taki problem:
Jeżeli definujecie pola, zwróćcie uwagę na to, czy pole to jest obowiązkowe, czy też opcjonalne. Pola takie jak username, password, confirmPassword powinny być required, ze swaggera wynika, że są opcjonalne aktualnie

A opcjonalnych nie mamy zadnych bo zadnych defaultow nie ma

Natalia dała taki problem: Jeżeli definujecie pola, zwróćcie uwagę na to, czy pole to jest obowiązkowe, czy też opcjonalne. Pola takie jak username, password, confirmPassword powinny być required, ze swaggera wynika, że są opcjonalne aktualnie A opcjonalnych nie mamy zadnych bo zadnych defaultow nie ma
s426135 requested review from s426134 2020-12-15 15:59:02 +01:00
Collaborator

chodzi o to, ze np. na rejestracji widać na swaggerze, które pola musimy podać:
image

Natomiast przy logowaniu:
image

Nie ma nigdzie , więc ednpoint wskazuje na to, że zadziała poprawnie również dla np podania userName null. Swagger domyślnie traktuje wszystkie paramtery jako optional, więc jeśli coś jest wymagane do podania to zaznaczamy to [required]

chodzi o to, ze np. na rejestracji widać na swaggerze, które pola musimy podać: ![image](/attachments/8062285d-46cf-4dd1-ad7b-a51b81896a26) Natomiast przy logowaniu: ![image](/attachments/600bcbd4-697b-4ee3-808e-ad05aabcb39a) Nie ma nigdzie , więc ednpoint wskazuje na to, że zadziała poprawnie również dla np podania userName null. Swagger domyślnie traktuje wszystkie paramtery jako optional, więc jeśli coś jest wymagane do podania to zaznaczamy to [required]
7.7 KiB
6.5 KiB
s426134 merged commit c384b279cb into dev 2020-12-16 21:49:35 +01:00
s426135 deleted branch SES-92 2020-12-16 21:51:01 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: s426135/session-companion#30
No description provided.