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

SES-118 Endpoint dla staystyk #50

Merged
s426134 merged 11 commits from SES-118 into dev 2021-01-04 09:40:04 +01:00
Conversation 0 Commits 11 Files Changed 15 +10578
s426135 commented 2021-01-03 19:50:31 +01:00
Owner
Copy Link
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
SessionCompanion/SessionCompanion/Controllers/CharacterController.cs Outdated
@ -79,0 +79,4 @@
/// <summary>
/// Metoda zwraca wszystkie statystyki dla danej postaci
/// </summary>
/// <param name="characterId"></param>
s426134 commented 2021-01-03 22:38:08 +01:00
Collaborator
Copy Link

brak opisu

brak opisu
s426135 marked this conversation as resolved
SessionCompanion/SessionCompanion/Controllers/CharacterController.cs Outdated
@ -79,0 +80,4 @@
/// Metoda zwraca wszystkie statystyki dla danej postaci
/// </summary>
/// <param name="characterId"></param>
/// <returns></returns>
s426134 commented 2021-01-03 22:38:21 +01:00
Collaborator
Copy Link

brak opisu

brak opisu
SessionCompanion/SessionCompanion/Controllers/CharacterController.cs Outdated
@ -79,0 +82,4 @@
/// <param name="characterId"></param>
/// <returns></returns>
[HttpGet("characterStats")]
public async Task<Either<CharacterEveryStatViewModel, ErrorResponse>> GetCharacterEveryStat(int characterId)
s426134 commented 2021-01-03 22:38:48 +01:00
Collaborator
Copy Link

brak required

brak required
s426135 marked this conversation as resolved
SessionCompanion/SessionCompanion/Controllers/CharacterController.cs Outdated
@ -79,0 +84,4 @@
[HttpGet("characterStats")]
public async Task<Either<CharacterEveryStatViewModel, ErrorResponse>> GetCharacterEveryStat(int characterId)
{
var statistics = await _service.GetCharacterEveryStat(characterId);
s426134 commented 2021-01-03 22:40:24 +01:00
Collaborator
Copy Link

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.
Reviewers
No reviewers
s426134
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
s426135
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.
Delete Branch "SES-118"

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?