Merge branch 'offset-message' of github.com:Aiven-Open/klaw into offs…

…et-message
Aiven-Open · Nov 23, 2023 · 658aa69 · 658aa69
2 parents d8d799b + 5d263ec
commit 658aa69
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/coral/src/app/features/topics/details/messages/useMessagesFilters.test.tsx b/coral/src/app/features/topics/details/messages/useMessagesFilters.test.tsx
@@ -243,7 +243,8 @@ describe("useMessagesFilters.tsx", () => {
       expect(isValid).toBe(false);
 
       expect(result.current.filterErrors).toStrictEqual({
-        customOffsetFilters: "Please enter an offset",
+        customOffsetFilters:
+          "Please enter the number of recent offsets you want to view",
         partitionIdFilters: null,
       });
     });
@@ -269,7 +270,8 @@ describe("useMessagesFilters.tsx", () => {
       expect(isValid).toBe(false);
 
       expect(result.current.filterErrors).toStrictEqual({
-        customOffsetFilters: "Max offset: 100",
+        customOffsetFilters:
+          "Entered value exceeds the view limit for offsets: 100",
         partitionIdFilters: null,
       });
     });

diff --git a/coral/src/app/features/topics/details/messages/useMessagesFilters.tsx b/coral/src/app/features/topics/details/messages/useMessagesFilters.tsx
@@ -65,9 +65,9 @@ function useMessagesFilters(): OffsetFilters {
         : null;
     const customOffsetFiltersError =
       getCustomOffset() === "" || getCustomOffset() === null
-        ? "Please enter an offset"
+        ? "Please enter the number of recent offsets you want to view"
         : Number(getCustomOffset()) > 100
-        ? "Max offset: 100"
+        ? "Entered value exceeds the view limit for offsets: 100"
         : null;
 
     setFilterErrors({
@@ -117,7 +117,8 @@ function useMessagesFilters(): OffsetFilters {
     if (customOffset.length === 0) {
       setFilterErrors((prev) => ({
         ...prev,
-        customOffsetFilters: "Please enter an offset",
+        customOffsetFilters:
+          "Please enter the number of recent offsets you want to view",
       }));
       deleteCustomOffset();
       return;
@@ -126,9 +127,9 @@ function useMessagesFilters(): OffsetFilters {
       ...prev,
       customOffsetFilters:
         customOffset === ""
-          ? "Please enter an offset"
+          ? "Please enter the number of recent offsets you want to view"
           : Number(customOffset) > 100
-          ? "Max offset: 100"
+          ? "Entered value exceeds the view limit for offsets: 100"
           : null,
     }));
     searchParams.set(NAMES.customOffset, customOffset);