- Issue created by @brianperry
- Status changed to Postponed
over 1 year ago 9:33pm 26 August 2023 - 🇺🇸United States brianperry
Marking this as postponed as we focus on the vertical slice POC.
- Status changed to Active
about 1 year ago 2:29pm 1 November 2023 - 🇺🇸United States brianperry
For 1.0 my guess is that this is what we'll need to focus on:
> Serialization could be relevant when we support non-read operations in 1.0.
- Assigned to brianperry
- 🇺🇸United States brianperry
Picking this one up. I'm going to first do some experimentation now that our CRUD methods exist to determine if a serializer would actually be useful. I'll then update the issue accordingly.
- 🇺🇸United States brianperry
Archiving previous thoughts:
Starting this as a clearing ground for post MVP serializer related thoughts and will convert this to a more organized issue later.
Assumptions:
* For the POC, really what we were after is a deserialization - converting a JSON:API response into a simpler JS object.
* Serialization could be relevant when we support non-read operations in 1.0.
* Deserialization is a common issue for JSON:API responses, but less so for GraphQL.Questions:
* Should the JSON:API client deserialize by default? Current thought is no.
* Should the serializer option be part of the base class, or the JsonApiClient Class? On the fence about this, but leaning base class.
* Should the JSON:API client ship with a serializer/deserializer? If yes, should we adopt a dependency, or should we create our own?Libraries
* Jsona https://github.com/olosegres/jsona
* jsonapi-serializer https://github.com/SeyZ/jsonapi-serializer
* js-japi https://github.com/mathematic-inc/ts-japi (serializer only)
* deserialize-json-api https://github.com/weilandia/deserialize-json-api - Merge request !48Update ApiClient Serializer Type to Include serialize method → (Merged) created by brianperry
- Issue was unassigned.
- Status changed to Needs review
10 months ago 3:30pm 18 February 2024 - Status changed to RTBC
10 months ago 12:25am 19 February 2024 - Status changed to Fixed
10 months ago 12:59pm 19 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.