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
fix(ivy): don't project removed placeholders with runtime i18n #30783
fix(ivy): don't project removed placeholders with runtime i18n #30783
Conversation
5006e98
to
9ea1bc9
Compare
7985073
to
70feb6a
Compare
When translated content was projected, all of the content was reappended, even the placeholders that had been removed in the translation. To avoid that we added a new flag on `TNode` that specifies that a node is detached, in which case it should be ignored by the projection. FW-1319 #resolve
70feb6a
to
f121351
Compare
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.
👍 for compiler (no compiler changes anymore)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
When translated content was projected, all of the content was reappended, even the placeholders that had been removed in the translation.
To avoid that we added a new flag on
TNode
that specifies that a node is detached, in which case it should be ignored by the projection.Ignore the first 3 commits that belong to the PRs #30696 and #30782. It should only be merged after these two.