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

Remove jsonrecords Twig helper #271

Closed
JarJak opened this issue Jan 17, 2019 · 2 comments

Comments

@JarJak
Copy link
Member

commented Jan 17, 2019

Content serialization should be done with Serializable or JsonSerializable interface. JSONed Content should look the same in Twig as well as in API response.

So:

{{ jsonrecords(records) }}

Should be replaced with:

{{ records|json_encode }}
@JarJak

This comment has been minimized.

Copy link
Member Author

commented Jan 21, 2019

Related to #150 as this is what we get after Content serialization

@JarJak

This comment has been minimized.

Copy link
Member Author

commented Feb 7, 2019

Will get removed with #151

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.