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 +16 -2
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 146abad9f5 - Show all commits

18
SessionCompanion/SessionCompanion/Controllers/SpellController.cs
View File

@ -1,5 +1,7 @@
using Microsoft.AspNetCore.Mvc;
using SessionCompanion.Extensions.EitherType;
using SessionCompanion.Services.Interfaces;
using SessionCompanion.ViewModels.ApiResponses;
using SessionCompanion.ViewModels.SpellViewModels;
using System;
using System.Collections.Generic;
@ -20,9 +22,21 @@ namespace SessionCompanion.Controllers
/// </summary>
/// <returns>Lista wszystkich zaklęć w bazie danych</returns>
s426135 marked this conversation as resolved
s426134 commented 2021-01-08 14:22:15 +01:00
Review
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.
[HttpGet("getAllSpells")]
public async Task<List<SpellViewModel>> GetSpells()
public async Task<Either<List<SpellViewModel>, ErrorResponse>> GetSpells()
{
return _service.Get().ToList();
try
{
var spells = _service.Get().ToList();
return spells;
}
catch (Exception e)
{
return new ErrorResponse()
{
StatusCode = 204,
Message = e.Message
};
}
}
}
}