Alv-no / Alv-no/alvtime

Ta i bruk korrekte HTTP-statuskoder for endepunkter

Aberta
#178 4 comentários 0 reações 0 responsáveis Ver no GitHub
.NET good first issue
Linguagem predominante
C#
Estrelas
9
Forks
5
Merge médio
12h 2min
PRs com merge (30d)
5

Descrição

Per nå returnerer alle endepunkt 200 OK. Undersøk om det går an å returnere andre koder avhengig av resultat. F.eks. 201 Created når noe lages, eller 204 No Content for deleted. Bør gi annen HTTP-kode hvis man f.eks. prøver å føre time på en locked task.

For mange av endepunktene tas det inn en liste og returneres en liste. For eksempel føre timer tar inn en liste så gjøres for each på hver av timene man skal føre. Problemet her er at dersom man sender inn en liste med 5 entries man vil føre, og de 4 første går fint, men den siste feiler, så vil de 4 første være lagret og alt ok med dem, men du får likevel returnert en 500 fordi den siste feilet. Kanskje man burde wrappe hele dette i en transaction så HTTP koden gjenspeiler hele operasjonen.

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.