-
Notifications
You must be signed in to change notification settings - Fork 3
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
EXT-1434: Removing data.language
.
#148
Conversation
BREAKING CHANGE: `data.story.language` is now used to access the language of the story.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@eunjae-lee @BibiSebi the build fails. I guess this is because |
Yeah can you revert the version changes? Let's just remove data.language in this PR, and deploy the library and the CLI separately. Let me guide you through that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The content itself looks good except for the versions to be reverted
# Conflicts: # packages/field-plugin/src/messaging/pluginMessage/containerToPluginMessage/StoryData.test.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
What?
Removing
data.language
from the field plugin data.This is a breaking change that requires the library version to be bumped, which in turn requires the templates to be updated and the CLI version to be bumped.
As a replacement of
data.language
, usedata.story.lang
.Why?
When introduced, the thought was that this property contains the language of the Storyblok application as configured by the user's preferences. Instead, it turns out that this contains the content language. As such, use
data.story.lang
as a replacement.