[DOP-22993] fix after review

src/shared/ui/CronSelect/components/DynamicSelect/index.tsx

@@ -1,6 +1,7 @@
 import React from 'react';
 import { Period } from '@shared/services';
 import { Select } from 'antd';
+import { getOrdinalNumber } from '@shared/utils';
 
 import classes from '../../styles.module.less';
 import { DAYS_OF_MONTH_SELECT_OPTIONS, DAYS_OF_WEEK_SELECT_OPTIONS } from '../../constants';
@@ -27,13 +28,16 @@ export const DynamicSelect = ({
       );
     case Period.MONTH:
       return (
-        <Select
-          className={classes.date}
-          size="large"
-          onChange={onChangeMonthDay}
-          options={DAYS_OF_MONTH_SELECT_OPTIONS}
-          value={monthDay}
-        />
+        <div className={classes.month}>
+          <Select
+            className={classes.date}
+            size="large"
+            onChange={onChangeMonthDay}
+            options={DAYS_OF_MONTH_SELECT_OPTIONS}
+            value={monthDay}
+          />
+          <span>{getOrdinalNumber(monthDay!, true)}</span>
+        </div>
       );
     default:
       return null;

src/shared/ui/CronSelect/hooks/useCron/useCron.ts

@@ -22,7 +22,7 @@ export const useCron = ({ value, onChange = () => undefined }: UseCronProps) =>
         cronService.setMonthDay(dayjs().date());
         break;
     }
-    onChange(cronService.toString());
+    handleChange();
   };
 
   const handleChangeWeekDay = (weekDay: CronSegmentValue) => {

src/shared/ui/CronSelect/styles.module.less

@@ -8,6 +8,12 @@
     align-items: center;
     gap: 8px;
 
+    .month {
+      display: flex;
+      align-items: center;
+      gap: 2px;
+    }
+
     .period {
       width: 100px;
     }

src/shared/utils/getOrdinalNumber/index.ts

@@ -1,21 +1,29 @@
 /**
  * Util for getting number with its ordinal postfix
  *
- * @param value - number.
+ * @param value - number
+ * @param onlyPostfix - flag for returning only postfix without number
  *
  * @returns - ordinal number
  */
-export const getOrdinalNumber = (value: number): string => {
-  const currentValue = value.toString();
-  const valueLastChar = Number(currentValue[currentValue.length - 1]);
-  if (valueLastChar === 1) {
-    return `${value}st`;
+export const getOrdinalNumber = (value: number, onlyPostfix = false): string => {
+  const lastDigit = value % 10;
+  const twoLastDigits = value % 100;
+  let postfix = '';
+
+  if (lastDigit === 1 && twoLastDigits !== 11) {
+    postfix = 'st';
+  } else if (lastDigit === 2 && twoLastDigits !== 12) {
+    postfix = 'nd';
+  } else if (lastDigit === 3 && twoLastDigits !== 13) {
+    postfix = 'rd';
+  } else {
+    postfix = 'th';
   }
-  if (valueLastChar === 2) {
-    return `${value}nd`;
+
+  if (onlyPostfix) {
+    return postfix;
   }
-  if (valueLastChar === 3) {
-    return `${value}rd`;
-  }
-  return `${value}th`;
+
+  return `${value}${postfix}`;
 };