SES-118 Endpoint dla staystyk #50

Merged
s426134 merged 11 commits from SES-118 into dev 2021-01-04 09:40:04 +01:00
Owner
No description provided.
s426135 self-assigned this 2021-01-03 19:50:31 +01:00
s426135 changed title from SES-118 to SES-118 Endpoint dla staystyk 2021-01-03 19:50:41 +01:00
s426135 requested review from s426134 2021-01-03 19:50:44 +01:00
s426134 requested changes 2021-01-03 22:42:10 +01:00
@ -79,0 +79,4 @@
/// <summary>
/// Metoda zwraca wszystkie statystyki dla danej postaci
/// </summary>
/// <param name="characterId"></param>
Collaborator

brak opisu

brak opisu
s426135 marked this conversation as resolved
@ -79,0 +80,4 @@
/// Metoda zwraca wszystkie statystyki dla danej postaci
/// </summary>
/// <param name="characterId"></param>
/// <returns></returns>
Collaborator

brak opisu

brak opisu
@ -79,0 +82,4 @@
/// <param name="characterId"></param>
/// <returns></returns>
[HttpGet("characterStats")]
public async Task<Either<CharacterEveryStatViewModel, ErrorResponse>> GetCharacterEveryStat(int characterId)
Collaborator

brak required

brak required
s426135 marked this conversation as resolved
@ -79,0 +84,4 @@
[HttpGet("characterStats")]
public async Task<Either<CharacterEveryStatViewModel, ErrorResponse>> GetCharacterEveryStat(int characterId)
{
var statistics = await _service.GetCharacterEveryStat(characterId);
Collaborator

Statystyki są niezmienne w budowe, wystarczy pobrać każdą z nich z osobna lub jednym zapytaniem wsyzstkie i póxniej zbudować z silnego typowania dla każdego ability odpowiednio widok i to połączyć w listę statystyk.
Można też użyć do tego mappera, ale to trochę więcej zabawy. Kod bedzie na "wyższym poziomie" ale nie trzeba tego robić

Statystyki są niezmienne w budowe, wystarczy pobrać każdą z nich z osobna lub jednym zapytaniem wsyzstkie i póxniej zbudować z silnego typowania dla każdego ability odpowiednio widok i to połączyć w listę statystyk. Można też użyć do tego mappera, ale to trochę więcej zabawy. Kod bedzie na "wyższym poziomie" ale nie trzeba tego robić
s426135 requested review from s426134 2021-01-04 01:06:04 +01:00
s426134 merged commit 36b89d209f into dev 2021-01-04 09:40:04 +01:00
s426135 deleted branch SES-118 2021-01-04 15:50:21 +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#50
No description provided.