Skip to content

WIP: ToSchema + deriving-aeson#607

Closed
brprice wants to merge 2 commits intomainfrom
brprice/openapi-primerjson
Closed

WIP: ToSchema + deriving-aeson#607
brprice wants to merge 2 commits intomainfrom
brprice/openapi-primerjson

Conversation

@brprice
Copy link
Copy Markdown
Contributor

@brprice brprice commented Aug 3, 2022

WIP to kick CI
I should drop a diff of the openapi.json here

- the manual instance (look for library?)
- my choices around what things are derived via and what are gnd and
  what are plain (should match ToJSON instances)

I would like to brush off the testing work and ensure that catches all
possible bugs here!
TODO: check what difference this via-custom makes when have non Vanilla
(unsure if it currently is applied to any types where it is interesting)
TODO: check this via-custom is correct wrt the ToJSON encoding
@brprice
Copy link
Copy Markdown
Contributor Author

brprice commented Aug 12, 2022

Closing in favour of #626

@brprice brprice closed this Aug 12, 2022
@brprice brprice deleted the brprice/openapi-primerjson branch August 12, 2022 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Derive ToSchema instances via PrimerJSON

1 participant