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

Generate a JSONAPISerializer to stop relying on the defaultSerializer behavior #218

Closed
wants to merge 1 commit into from
Closed

Generate a JSONAPISerializer to stop relying on the defaultSerializer behavior #218

wants to merge 1 commit into from

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Feb 22, 2021

@mansona
Copy link
Member

mansona commented Feb 22, 2021

Hey thanks for the contribution 🎉 I was wondering if it were possible to add an application serializer to Guidemaker that would solve this issue? 🤔 Guidemaker is the core-addon that powers the cli-guides, as you can see there is no code in the Ember App in this repo (other than now your Application Serializer 😂) because it is all being provided by Guidemaker: https://github.com/empress/guidemaker

@Windvis
Copy link
Contributor Author

Windvis commented Feb 22, 2021

Yea sorry, I didn't really look into the details 😄. I just noticed the warnings in the console and "fixed" it without looking at the rest of the codebase. I just found this issue: empress/guidemaker#34 and a related PR which covers it already I think?

@Windvis Windvis closed this Feb 22, 2021
@Windvis Windvis deleted the chore/default-serializer-deprecation branch February 22, 2021 21:48
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.

None yet

2 participants