Merge branch 'FET-1689' of https://github.com/ensdomains/ens-app-v3 i…

…nto FET-1689

src/utils/string.test.ts

@@ -1,5 +1,6 @@
-import { it, describe, expect } from "vitest";
-import { parseNumericString } from "./string";
+import { describe, expect, it } from 'vitest'
+
+import { parseNumericString } from './string'
 
 describe('parseNumericString', () => {
   it('should return an integer', () => {
@@ -22,7 +23,7 @@ describe('parseNumericString', () => {
     expect(parseNumericString('-123')).toBe(null)
   })
 
-  it('should return null for a negative number', () => {
+  it('should return null for a invalid number', () => {
     expect(parseNumericString('1a23')).toBe(null)
   })
-})
+})

src/utils/string.ts

@@ -1,9 +1,11 @@
-export const parseNumericString = (time: string | null): number | null => {
-  if (!time) return null
+export const parseNumericString = (value: string | null): number | null => {
+  if (!value) return null
 
-  if (typeof +time === 'number' && !Number.isNaN(+time)) {
-    return +time
+  const parsed = Number(value)
+
+  if (typeof parsed !== 'number' || Number.isNaN(parsed) || parsed < 0) {
+    return null
   }
 
-  return null
+  return parseInt(String(parsed))
 }