SES-129 Endpoint zaklęć #53

Merged
s426134 merged 4 commits from SES-129 into dev 2021-01-09 14:09:33 +01:00
2 changed files with 176 additions and 134 deletions

View File

@ -0,0 +1,42 @@
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;
using System.Linq;
using System.Threading.Tasks;
namespace SessionCompanion.Controllers
{
[Route("api/spell")]
[ApiController]
public class SpellController : Controller
{
private readonly ISpellService _service;
public SpellController(ISpellService service) => _service = service;
/// <summary>
/// Metoda zwraca wszystkie dostępne zaklecia
/// </summary>
/// <returns>Lista wszystkich zaklęć w bazie danych</returns>
s426135 marked this conversation as resolved
Review

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<Either<List<SpellViewModel>, ErrorResponse>> GetSpells()
{
try
{
var spells = _service.Get().ToList();
return spells;
}
catch (Exception e)
{
return new ErrorResponse()
{
StatusCode = 204,
Message = e.Message
};
}
}
}
}