-
Notifications
You must be signed in to change notification settings - Fork 7
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
QA-15315 Backport from jcontent, clearing multiple localized props #1783
Conversation
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.
Probably need to update our jahia-depends clause to latest graphql snapshot (without the snapshot qualifier) https://github.com/Jahia/content-editor/blob/master/pom.xml#L52
c2eca7f
to
b313e59
Compare
@@ -22,6 +22,7 @@ | |||
- 'mvn:org.jahia.modules/dx-base-demo-components/2.4.0' | |||
- 'mvn:org.jahia.modules/digitall/2.3.0' | |||
- 'mvn:org.jahia.modules/jcontent/2.15.0' | |||
- 'mvn:org.jahia.modules/graphql-dxm-provider/2.22.0-SNAPSHOT' |
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.
It's ok with snapshot like that I think, but if I recall correctly it should take snapshot if you just pass 2.22.0, so when it's released we don't have to modify this file gain. You can run provisioning locally to verify, it's probably a good idea to update if it works that way.
JIRA
https://jira.jahia.org/browse/QA-15315
Description
Tests
The following are included in this PR
Checklist
I have considered the following implications of my change:
Documentation