-
Notifications
You must be signed in to change notification settings - Fork 10
[CRYSTAL-574] Fix: remove escaped quotes and standardize trigger parameters in validation messages #401
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
[CRYSTAL-574] Fix: remove escaped quotes and standardize trigger parameters in validation messages #401
Conversation
- Remove the \ prefix and suffix in strings wrapped in double quotes
66ed247 to
76b2c82
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.
Pull Request Overview
Fixes YAML formatting issues and parameter inconsistencies in Custom Objects V2 validation error messages to improve display quality and ensure consistent parameter usage.
- Removed escaped quotes that caused
\prefixes/suffixes to appear in error messages - Replaced
%{trigger_title}with%{trigger_key}as the standardized unique identifier for triggers - Created new translation entries with corrected formatting and marked obsolete entries with deprecation dates
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_cov2_object_reference_in_fields_v2" | ||
| title: 'App builder job: requirements file contains an invalid object reference in object_fields for Custom Objects V2. Do not translate "requirements.json", "object_fields", %{item_identifier}, %{object_key}.' | ||
| value: The requirements.json file contains an invalid object reference in object_fields. Field %{item_identifier} references object %{object_key} which does not exist in the objects array. |
Copilot
AI
Oct 16, 2025
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 value field is missing quotes around the string value, which is inconsistent with the YAML format used throughout the rest of the file.
| value: The requirements.json file contains an invalid object reference in object_fields. Field %{item_identifier} references object %{object_key} which does not exist in the objects array. | |
| value: "The requirements.json file contains an invalid object reference in object_fields. Field %{item_identifier} references object %{object_key} which does not exist in the objects array." |
| screenshot: "https://drive.google.com/file/d/1yHiq5uUyGmQjLgO72ARAVCgFA1tXFUHT/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_cov2_object_reference_in_triggers_v2" | ||
| title: 'App builder job: requirements file contains an invalid object reference in object_triggers for Custom Objects V2. Do not translate "requirements.json", "object_fields", %{item_identifier}, %{object_key}.' |
Copilot
AI
Oct 16, 2025
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 title references 'object_fields' in the translation note, but this validation message is for 'object_triggers'. The note should reference 'object_triggers' instead of 'object_fields'.
| title: 'App builder job: requirements file contains an invalid object reference in object_triggers for Custom Objects V2. Do not translate "requirements.json", "object_fields", %{item_identifier}, %{object_key}.' | |
| title: 'App builder job: requirements file contains an invalid object reference in object_triggers for Custom Objects V2. Do not translate "requirements.json", "object_triggers", %{item_identifier}, %{object_key}.' |
| screenshot: "https://drive.google.com/file/d/1peD9NjuFQ6_dW4ia6Cxz-GimTN0rdjMk/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.missing_cov2_trigger_schema_key_v2" | ||
| title: 'App builder job: requirements file is missing a custom object trigger schema key. These are the required keys: (title, actions, conditions, object_key). The term \"schema key\" typically refers to an identifier or attribute that defines a specific item within a schema and it should be translated. Leave requirements.json as is (do not translate)' |
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.
| title: 'App builder job: requirements file is missing a custom object trigger schema key. These are the required keys: (title, actions, conditions, object_key). The term \"schema key\" typically refers to an identifier or attribute that defines a specific item within a schema and it should be translated. Leave requirements.json as is (do not translate)' | |
| title: 'App builder job: requirements file is missing a custom object trigger schema key. These are the required keys: (title, actions, conditions, object_key). The term "schema key" typically refers to an identifier or attribute that defines a specific item within a schema and it should be translated. Leave requirements.json as is (do not translate)' |
| screenshot: "https://drive.google.com/file/d/1hTiBvSHqEXCsoa2YAt1bnGNCs8cXSzlx/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_cov2_trigger_actions_structure_v2" | ||
| title: "App builder job: requirements file contains an invalid custom object trigger actions structure. Actions must be an array. 'actions' is part of requirements file. Leave requirements.json and 'actions' as is (do not translate)" |
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.
| title: "App builder job: requirements file contains an invalid custom object trigger actions structure. Actions must be an array. 'actions' is part of requirements file. Leave requirements.json and 'actions' as is (do not translate)" | |
| title: "App builder job: requirements file contains an invalid custom object trigger actions structure. Actions must be an array. 'actions' is part of requirements file. Leave requirements.json as is (do not translate). 'actions' is part of the schema and shouldn't be translated (do not translate)." |
| screenshot: "https://drive.google.com/file/d/1UeBrY4GlnHDK19zuUE8xUoHnZl9HQv3y/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.empty_cov2_trigger_actions_v2" | ||
| title: "App builder job: requirements file contains empty custom object trigger actions. At least one action must be specified. Leave requirements.json as is (do not translate)" |
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.
| title: "App builder job: requirements file contains empty custom object trigger actions. At least one action must be specified. Leave requirements.json as is (do not translate)" | |
| title: "App builder job: requirements file contains empty custom object trigger actions. At least one action must be specified. Leave requirements.json as is (do not translate). 'actions' is part of the schema and shouldn't be translated (do not translate)." |
| screenshot: "https://drive.google.com/file/d/1ywYAYOoeVmjUgZ2yfFl3YdJ-xkrBSgUG/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_object_fields_structure_in_cov2_requirements_v2" | ||
| title: "App builder job: requirements file contains invalid object_fields structure for custom objects v2. Object_fields must be an array. Leave requirements.json and 'object_fields' as is (do not translate)" |
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.
| title: "App builder job: requirements file contains invalid object_fields structure for custom objects v2. Object_fields must be an array. Leave requirements.json and 'object_fields' as is (do not translate)" | |
| title: "App builder job: requirements file contains invalid object_fields structure for custom objects v2. Object_fields must be an array. Leave requirements.json as is (do not translate). 'object_fields' is part of the schema and shouldn't be translated (do not translate). " |
| screenshot: "https://drive.google.com/file/d/1ishzANlbt4PiyxikwXDTTPKy9drlWWlP/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_cov2_requirements_structure_v2" | ||
| title: 'App builder job: requirements file contains invalid custom objects v2 structure. An "object" here refers to a programming object (a set of key-value pairs), not a custom object in Zendesk. Leave requirements.json, custom_objects_v2 as is (do not translate)' |
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.
| title: 'App builder job: requirements file contains invalid custom objects v2 structure. An "object" here refers to a programming object (a set of key-value pairs), not a custom object in Zendesk. Leave requirements.json, custom_objects_v2 as is (do not translate)' | |
| title: 'App builder job: requirements file contains invalid custom objects v2 structure. An "object" here refers to a programming object (a set of key-value pairs), not a custom object in Zendesk. Leave 'requirements.json' as is (do not translate). 'custom_objects_v2' is part of the schema and shouldn't be translated (do not translate).' |
| screenshot: "https://drive.google.com/file/d/1TQSnKbBXxLSYFpcgEBGNceLpo5ARMBAa/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_object_triggers_structure_in_cov2_requirements_v2" | ||
| title: "App builder job: requirements file contains invalid object_triggers structure for custom objects v2. Object_triggers must be an array. Leave requirements.json and 'object_triggers' as is (do not translate)" |
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.
| title: "App builder job: requirements file contains invalid object_triggers structure for custom objects v2. Object_triggers must be an array. Leave requirements.json and 'object_triggers' as is (do not translate)" | |
| title: "App builder job: requirements file contains invalid object_triggers structure for custom objects v2. Object_triggers must be an array. Leave requirements.json as is (do not translate). 'object_triggers' is part of the schema and shouldn't be translated (do not translate)." |
| screenshot: "https://drive.google.com/file/d/19r2xv15OnXtu2-JxU_ioAVMmaP33DiB-/view?usp=sharing" | ||
| - translation: | ||
| key: "txt.apps.admin.error.app_build.invalid_cov2_object_reference_in_fields_v2" | ||
| title: 'App builder job: requirements file contains an invalid object reference in object_fields for Custom Objects V2. Do not translate "requirements.json", "object_fields", %{item_identifier}, %{object_key}.' |
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.
| title: 'App builder job: requirements file contains an invalid object reference in object_fields for Custom Objects V2. Do not translate "requirements.json", "object_fields", %{item_identifier}, %{object_key}.' | |
| title: 'App builder job: requirements file contains an invalid object reference in object_fields for Custom Objects V2. Do not translate "requirements.json", "object_fields" as this is a schema key, %{item_identifier}, %{object_key}.' |
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.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
zenfw
left a comment
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.
LGTM
💐
/cc @zendesk/wattle
Description
Fixes YAML formatting issues and parameter inconsistencies in Custom Objects V2 validation error messages.
Key Changes:
""wrapper quotes that caused\prefix/suffix to appear in error messages%{trigger_title}with%{trigger_key}as unique identifierReferences
JIRA: https://zendesk.atlassian.net/browse/CRYSTAL-574
Before merging this PR
Risks