Browse Source

fixing error messages to be user friendly (#2049)

Robert Azizbekyan 3 years ago
parent
commit
50afb26f95
1 changed files with 24 additions and 5 deletions
  1. 24 5
      kafka-ui-react-app/src/lib/yupExtended.ts

+ 24 - 5
kafka-ui-react-app/src/lib/yupExtended.ts

@@ -52,13 +52,32 @@ export const topicFormValidationSchema = yup.object().shape({
       TOPIC_NAME_VALIDATION_PATTERN,
       TOPIC_NAME_VALIDATION_PATTERN,
       'Only alphanumeric, _, -, and . allowed'
       'Only alphanumeric, _, -, and . allowed'
     ),
     ),
-  partitions: yup.number().min(1).required(),
-  replicationFactor: yup.number().min(1).required(),
-  minInsyncReplicas: yup.number().min(1).required(),
+  partitions: yup
+    .number()
+    .min(1)
+    .required()
+    .typeError('Number of partitions is required and must be a number'),
+  replicationFactor: yup
+    .number()
+    .min(1)
+    .required()
+    .typeError('Replication factor is required and must be a number'),
+  minInsyncReplicas: yup
+    .number()
+    .min(1)
+    .required()
+    .typeError('Min in sync replicas is required and must be a number'),
   cleanupPolicy: yup.string().required(),
   cleanupPolicy: yup.string().required(),
-  retentionMs: yup.number().min(-1, 'Must be greater than or equal to -1'),
+  retentionMs: yup
+    .number()
+    .min(-1, 'Must be greater than or equal to -1')
+    .typeError('Time to retain data is required and must be a number'),
   retentionBytes: yup.number(),
   retentionBytes: yup.number(),
-  maxMessageBytes: yup.number().min(1).required(),
+  maxMessageBytes: yup
+    .number()
+    .min(1)
+    .required()
+    .typeError('Maximum message size is required and must be a number'),
   customParams: yup.array().of(
   customParams: yup.array().of(
     yup.object().shape({
     yup.object().shape({
       name: yup.string().required('Custom parameter is required'),
       name: yup.string().required('Custom parameter is required'),