Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Json api helpers #289

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Json api helpers #289

wants to merge 11 commits into from

Conversation

moxley
Copy link
Contributor

@moxley moxley commented Sep 15, 2020

No description provided.

@moxley moxley mentioned this pull request Sep 15, 2020
Adds CastTests to ensure usability
Moves CartModul defintions into JsonApiSchemas Module
@scottmessinger
Copy link

I'm not sure if this is out of scope for this feature, but one thing I'd love is the unification of the module I use to generate the response and the module I use to specify the schema of the response. I've been using ja-serializer and would love if open_api_spex would let me specify the schema and use that same module to transform a struct into a JSON:API response. Again, I'm not sure if that's out of scope, but since this is still in the draft phase, I thought I'd mention it!

@mrmstn
Copy link
Contributor

mrmstn commented Jan 5, 2021

Hi @scottmessinger, Thank you very much for the info, this really sounds interesting!
As I've never worked with JaSerializer, I'll need to look into it a little bit more :)

@Nezteb
Copy link

Nezteb commented Sep 27, 2022

This is something I'm also very interested in, similar to phoenix_swagger's own implementation.

I'm currently working on an experiment to use beam-community/jsonapi alongside open_api_spex, though I think that'd be more work than it's worth at this point. If anyone else has any guidance on using the two libraries together, I'd be extremely grateful! 😄

@Nezteb
Copy link

Nezteb commented Oct 5, 2022

@moxley If there is any part of this PR you'd like assistance with (functionality, features, tests, docs, etc.) please let me know!

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.

4 participants