s426135/session-companion
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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
Conversation 1 Commits 1 Files Changed 3 +16 -9
s426135 commented 2020-12-15 15:58:56 +01:00
Owner
Copy Link

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
s426134 commented 2020-12-16 18:15:29 +01:00
Collaborator
Copy Link

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]
image.png
7.7 KiB
image.png
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.
Reviewers
No reviewers
s426134
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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.
Delete Branch "SES-92"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?