-
Notifications
You must be signed in to change notification settings - Fork 338
CJM-97211 Introduce xdm-schema for message-rendering-context dataset #1977
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
base: master
Are you sure you want to change the base?
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.
Please run the following commands on your local clone before creating the pull request:
npm install
npm run lint
npm test
Fix any issues then create the pull request.
"type": "object", | ||
"title": "Message Details", | ||
"description": "Contains the specific content details for the message, varying by channel (e.g., email or SMS).", | ||
"oneOf": [ |
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.
I am not sure if oneOf will work. What are you expecting the validator to validate?
"title": "Message Details", | ||
"description": "Contains the specific content details for the message, varying by channel (e.g., email or SMS).", | ||
"oneOf": [ | ||
{ |
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 structure of this JSON schema does not look right.
Please link to the issue #…