Skip to content

Commit 38815cf

Browse files
nanotaboadaCopilot
andcommitted
chore(rest): fix middleName null value and add CHANGELOG references (#493)
- Correct middleName from empty string to null in PUT request - Add issue reference to REST client CHANGELOG entries Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 75737b0 commit 38815cf

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

CHANGELOG.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@ This project uses famous football coaches as release codenames, following an A-Z
4444

4545
### Added
4646

47-
- `rest/players.rest` file covering all CRUD operations, compatible with the VS Code REST Client extension
48-
- `humao.rest-client` added to `.vscode/extensions.json` recommendations
47+
- `rest/players.rest` file covering all CRUD operations, compatible with the VS Code REST Client extension (#493)
48+
- `humao.rest-client` added to `.vscode/extensions.json` recommendations (#493)
4949
- UUID v4 primary key for the `players` table, replacing the previous integer PK (#66)
5050
- `PlayerRequestModel` Pydantic model for POST/PUT request bodies (no `id` field) (#66)
5151
- `PlayerResponseModel` Pydantic model for GET/POST response bodies (includes `id: UUID`) (#66)
@@ -71,7 +71,7 @@ This project uses famous football coaches as release codenames, following an A-Z
7171

7272
### Removed
7373

74-
- `postman_collections/` directory replaced by `rest/players.rest`
74+
- `postman_collections/` directory replaced by `rest/players.rest` (#493)
7575

7676
### Fixed
7777

rest/players.rest

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ Content-Type: application/json
7878

7979
{
8080
"firstName": "Emiliano",
81-
"middleName": "",
81+
"middleName": null,
8282
"lastName": "Martínez",
8383
"dateOfBirth": "1992-09-02T00:00:00.000Z",
8484
"squadNumber": 23,

0 commit comments

Comments
 (0)