Skip to content
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

Use the correct localization keys for core property validation errors #15931

Merged
merged 1 commit into from
Mar 22, 2024

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Mar 22, 2024

Description

When performing document/media/member property validation, there are three core validation errors that can occur:

  • A mandatory value was not supplied.
  • A mandatory value was empty.
  • A value did not match the configured validation (regex) pattern.

Per property one can configure specific validation error messages, but by default these are blank - which means we use error messages defined in the core instead.

Since these core error messages are localized in the client, the server has to return a localization key. And as of right now, the localization keys returned are wrong 😆 so this PR fixes the localization keys returned by the server to match those known by the client.

Copy link
Contributor

@iOvergaard iOvergaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks correct, we have these keys in the client!

@kjac kjac merged commit 0ed6567 into v14/dev Mar 22, 2024
16 checks passed
@kjac kjac deleted the v14/fix/core-validation-error-keys branch March 22, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants