-
Notifications
You must be signed in to change notification settings - Fork 5
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
IBX-4927: Changing custom tag configuration breaks existing Rich Text fields that include this tag #76
Conversation
updateInput(initialData); | ||
|
||
this.editor.model.document.on('change:data', () => { | ||
const data = this.getData(); |
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.
Is this.editor.getData(); synchronous operation?
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.
Yes. Also that's exact line from https://github.com/ibexa/product-catalog/pull/913 , only indentation was changed :)
25afe96
to
cfec74b
Compare
cfec74b
to
37600bb
Compare
@GrabowskiM After applying patch im getting no error after editing custom tag (which has deleted style section) Is this the intention of this pr? |
… fields that include this tag
37600bb
to
1e22a8b
Compare
SonarCloud Quality Gate failed.
|
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.
QA approved on IbexaDXP 4.4 commerce.
TODO:
$ composer fix-cs
).