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

SES-129 Endpoint zaklęć #53

Merged
s426134 merged 4 commits from SES-129 into dev 2021-01-09 14:09:33 +01:00
Conversation 0 Commits 4 Files Changed 2 +176 -134
s426135 commented 2021-01-07 21:26:27 +01:00
Owner
Copy Link
No description provided.
s426135 requested review from s426134 2021-01-07 21:26:31 +01:00
s426134 requested changes 2021-01-08 14:22:21 +01:00
SessionCompanion/SessionCompanion/Controllers/SpellController.cs
@ -0,0 +20,4 @@
/// </summary>
/// <returns>Lista wszystkich zaklęć w bazie danych</returns>
[HttpGet("getAllSpells")]
public async Task<List<SpellViewModel>> GetSpells()
s426134 commented 2021-01-08 14:22:15 +01:00
Collaborator
Copy Link

brakuje możliwości ErrorResponse. Pomimo tego, ze sama metoda teoretycznie nie ma sie gdzie wywalić (ściaga wszystko i przekazuje) to coś może gdzieś na serwerze pójść nie tak. zostanie zwrócony błąd i front się wywróci w konsoli.

brakuje możliwości ErrorResponse. Pomimo tego, ze sama metoda teoretycznie nie ma sie gdzie wywalić (ściaga wszystko i przekazuje) to coś może gdzieś na serwerze pójść nie tak. zostanie zwrócony błąd i front się wywróci w konsoli.
s426135 marked this conversation as resolved
s426135 requested review from s426134 2021-01-08 17:49:59 +01:00
s426134 merged commit 046ba41ebb into dev 2021-01-09 14:09:33 +01:00
s426134 deleted branch SES-129 2021-01-09 14:09:36 +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#53
No description provided.
Delete Branch "SES-129"

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?