Clean up absoluteUrl

Author: vcua-mobify

packages/template-retail-react-app/app/utils/url.js

@@ -13,26 +13,6 @@ import {
     getSiteByReference
 } from '@salesforce/retail-react-app/app/utils/site-utils'
 import {HOME_HREF, urlPartPositions} from '@salesforce/retail-react-app/app/constants'
-import {isAbsoluteURL} from '@salesforce/retail-react-app/app/page-designer/utils'
-
-/**
- * Constructs an absolute URL from a given path and an optional application origin.
- * TODO: How should this work with base path?
- *
- * @param {string} path - The relative URL path to be appended to the origin.
- * @param {string} [appOrigin] - The optional application origin (e.g., "https://example.com").
- *                                If not provided, the function will call `getAppOrigin()`.
- * @returns {string} - The fully qualified URL as a string.
- */
-export const absoluteUrl = (path, appOrigin) => {
-    // If path is not provided or already an absolute URL, return it as-is
-    if (!path || isAbsoluteURL(path)) {
-        return path
-    }
-
-    // absoluteUrl is not a react hook so we cannot use the useAppOrigin hook here
-    return new URL(path, appOrigin || getAppOrigin()).toString()
-}
 
 /**
  * Modifies a given url by adding/updating query parameters.

packages/template-retail-react-app/app/utils/url.test.js

@@ -13,7 +13,6 @@ import {
     getPathWithLocale,
     rebuildPathWithParams,
     removeQueryParamsFromPath,
-    absoluteUrl,
     createUrlTemplate,
     removeSiteLocaleFromPath,
     serverSafeEncode
@@ -394,36 +393,6 @@ describe('removeQueryParamsFromPath test', () => {
     })
 })
 
-describe('absoluteUrl', function () {
-    test('return undefined when path is not provided', () => {
-        const url = absoluteUrl()
-        expect(url).toBeUndefined()
-    })
-
-    test('return expected when path is a relative url', () => {
-        const url = absoluteUrl('/uk/en/women/dresses')
-        expect(url).toBe('https://www.example.com/uk/en/women/dresses')
-    })
-
-    test('return expected when path is an absolute url', () => {
-        const url = absoluteUrl('https://www.example.com/uk/en/women/dresses')
-        expect(url).toBe('https://www.example.com/uk/en/women/dresses')
-    })
-
-    test('return expected when path is a relative url and appOrigin is provided', () => {
-        const url = absoluteUrl('uk/en/women/dresses', 'https://www.custom.com')
-        expect(url).toBe('https://www.custom.com/uk/en/women/dresses')
-    })
-
-    test('return expected when path is an absolute url and appOrigin is provided', () => {
-        const url = absoluteUrl(
-            'https://www.example.com/uk/en/women/dresses',
-            'https://www.should-be-ignored.com'
-        )
-        expect(url).toBe('https://www.example.com/uk/en/women/dresses')
-    })
-})
-
 describe('removeSiteLocaleFromPath', function () {
     test('return path without site alias and locale', () => {
         const pathName = removeSiteLocaleFromPath('/uk/en-GB/account/wishlist')