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

feat(rendercontenttype): include "sys" with content type ID in the generated interface #4

Merged
merged 2 commits into from
Oct 9, 2019

Conversation

stevenpetryk
Copy link
Contributor

@stevenpetryk stevenpetryk commented Aug 22, 2019

It's helpful to be able to inspect an entry's sys.contentType.sys.id field to see what type it is,
especially when a link can be one of many types. This makes the types support such a check.

BREAKING CHANGE: Users who use one content type's interface to accept another content type in a function (for example, if one content type is a superset of another one) will need to use a union
type and allow all possible content types now.

Steven Petryk added 2 commits August 22, 2019 12:51
…nerated interface

It's helpful to be able to inspect an entry's `sys.contentType.sys.id` field to see what type it is,
especially when a link can be one of many types. This makes the types support such a check.

BREAKING CHANGE: Users who use one content type's interface to accept another content type in a
function (for example, if one content type is a superset of another one) will need to use a union
type and allow all possible content types now.
`)
type CONTENTFUL_DEFAULT_LOCALE_CODE = \\"en-US\\"
"
`)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what caused jest to start rendering nicely-indented inline snapshots, but I'm a big fan.

@stevenpetryk stevenpetryk merged commit d3b9c44 into master Oct 9, 2019
@stevenpetryk stevenpetryk deleted the sgp/include-sys-in-type branch October 9, 2019 04:55
@stevenpetryk
Copy link
Contributor Author

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant