Skip to content

Commit

Permalink
Update docs/adr/20240605-use-minimal-api-typed-results.md
Browse files Browse the repository at this point in the history
Co-authored-by: William Liebenberg [SSW • Microsoft MVP] <[email protected]>
  • Loading branch information
danielmackay and william-liebenberg authored Jul 2, 2024
1 parent be8c4db commit b229441
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/adr/20240605-use-minimal-api-typed-results.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Technical Story: https://github.com/SSWConsulting/SSW.CleanArchitecture/issues/1

## Context and Problem Statement

Currently, are APIs use the `Result` pattern to return a `Result<T>` object. This works OK, but additional metadata is required to determine the status code of the response. We need to add the metadata manually which means there is a chance of inconsistency between the response and the status code.
Currently, our APIs use the `Result` pattern to return a `Result<T>` object. This works OK, but additional metadata is required to determine the status code of the response. We need to add the metadata manually which means there is a chance of inconsistency between the response and the status code.

We have tried to get around this by using helper methods to create the consistent OpenAI metadata for each REST verb. This was an improvement, but we still have the problem of the API status codes not being consistent with the response.

Expand Down

0 comments on commit b229441

Please sign in to comment.