Skip to content

The contentFormatter for ContentType:Json does not correctly format strings #176

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.

Sign up for GitHub

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

Jump to bottom
Closed
Styn opened this issue Feb 19, 2021 · 5 comments · Fixed by #178
Closed

The contentFormatter for ContentType:Json does not correctly format strings #176

Styn opened this issue Feb 19, 2021 · 5 comments · Fixed by #178
Assignees
@Styn
Labels
bug Something isn't working next release

Comments

@Styn
Copy link
Contributor

Styn commented Feb 19, 2021

When sending a string as json, it should by json.stringified, otherwise the backend is not able to parse it correctly.

I'll create a PR for this
@js2me
Copy link
Member

js2me commented Feb 19, 2021

Hello @Styn ! Thanks for your contribution !

@js2me js2me added bug Something isn't working next release labels Feb 19, 2021
@js2me
Copy link
Member

js2me commented Feb 19, 2021

@Styn please, use the next branch!

@Styn
Copy link
Contributor Author

Styn commented Feb 19, 2021

Oh sorry, I already created it, will try to fix

@Styn Styn mentioned this issue Feb 19, 2021
Merged
@Styn
Copy link
Contributor Author

Styn commented Feb 19, 2021

My previous commits seem to also be included in that PR, even though I created a new fork...

@js2me js2me mentioned this issue Feb 19, 2021
Merged
@js2me js2me linked a pull request Feb 19, 2021 that will close this issue
Release 5.1.6 (Bugfix) #178
Merged
@js2me
Copy link
Member

js2me commented Feb 19, 2021

Fixed in 5.1.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Styn Styn

Labels
bug Something isn't working next release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Release 5.1.6 (Bugfix) acacode/swagger-typescript-api
2 participants
@Styn @js2me