cotti/guestbooky
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Turns out it answers 204 in production, so let's fix that. Also, let's improve example.html's response handling.

Browse source
This commit is contained in:
Felipe Cotti 2024-10-05 10:42:33 -03:00
parent 37f94ed1db
commit 8e164be0d7
2 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/Guestbooky/Guestbooky.API/Controllers/MessageController.cs
View file

@ -23,7 +23,7 @@ public class MessageController : ControllerBase
} }
[HttpPost] [HttpPost]
[ProducesResponseType(StatusCodes.Status201Created)] [ProducesResponseType(StatusCodes.Status204NoContent)]
[ProducesResponseType(typeof(ProblemDetails), StatusCodes.Status403Forbidden)] [ProducesResponseType(typeof(ProblemDetails), StatusCodes.Status403Forbidden)]
[ProducesResponseType(typeof(ProblemDetails), StatusCodes.Status500InternalServerError)] [ProducesResponseType(typeof(ProblemDetails), StatusCodes.Status500InternalServerError)]
public async Task<IActionResult> Insert([FromBody] InsertMessageRequestDto message, CancellationToken token) public async Task<IActionResult> Insert([FromBody] InsertMessageRequestDto message, CancellationToken token)

7
src/input-example/example.html
View file

@ -141,13 +141,12 @@ document
body: JSON.stringify(data), body: JSON.stringify(data),
}); });
const result = await response.json(); if (response.status >= 200 && response.status <= 299) {
if (response.ok) {
alert("Note added successfully!"); alert("Note added successfully!");
this.reset(); // Clear the form fields this.reset(); // Clear the form fields
} else { } else {
alert(result.errorMessage); const problem = await response.json();
alert(problem);
} }
}); });
</script> </script>