SES-105 list of logged charcters #36

Merged
s426134 merged 7 commits from SES-105 into dev 2020-12-21 13:40:47 +01:00
Owner
No description provided.
s426135 self-assigned this 2020-12-20 15:46:38 +01:00
s426135 requested review from s426134 2020-12-20 15:46:44 +01:00
s426135 changed title from SES-105 to SES-105 list of logged charcters 2020-12-20 15:48:21 +01:00
s426134 requested changes 2020-12-20 16:07:37 +01:00
@ -64,0 +74,4 @@
List<CharacterBasicStatsViewModel> characters = new List<CharacterBasicStatsViewModel>();
var connectedCharacters = _sessionHubData.ConnectedCharacters_Prop;
foreach (var characterId in connectedCharacters.Values)
Collaborator

lepiej użyć metody .Contains od linq

lepiej użyć metody .Contains od linq
@ -64,0 +80,4 @@
if (characters.Any())
return characters;
return new ErrorResponse()
Collaborator

W sumie to brak zalogowanych postaci nie jest błędem, po prostu nie ma nic do wyświetlenia

W sumie to brak zalogowanych postaci nie jest błędem, po prostu nie ma nic do wyświetlenia
s426134 merged commit 070cafed58 into dev 2020-12-21 13:40:47 +01:00
s426134 deleted branch SES-105 2020-12-21 13:40:55 +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#36
No description provided.