no basePath for external links

Author: upreeti

src/ui/link/index.tsx

@@ -72,8 +72,9 @@ const Link = forwardRef<HTMLButtonElement | HTMLAnchorElement, LinkProps>(
               )) ||
             ''
           : to;
+    const IS_EXTERNAL_LINK_REGEX = /^(?:(http|https):\/\/)/;
     const staticBasePath =
-      href != null || typeof to === 'string' ? routeAttributes.basePath : '';
+      (href != null && !IS_EXTERNAL_LINK_REGEX.test(href)) || typeof to === 'string' ? routeAttributes.basePath : '';
 
     const triggerPrefetch = useCallback(() => {
       // ignore if async route not ready yet

src/ui/link/test.tsx

@@ -91,7 +91,7 @@ describe('<Link />', () => {
     );
   });
 
-  it('should push history with correct link for href when given basePath', () => {
+  it('should push history with correct link for relative href when given basePath', () => {
     renderInRouter(
       'my link',
       {
@@ -105,6 +105,20 @@ describe('<Link />', () => {
     );
   });
 
+  it('should not add basePath for external links', () => {
+    renderInRouter(
+      'my link',
+      {
+        href: 'https://www.atlassian.com/',
+      },
+      '/base'
+    );
+    expect(screen.getByRole('link', { name: 'my link' })).toHaveAttribute(
+      'href',
+      'https://www.atlassian.com/'
+    );
+  });
+
   it('should pass props to the child element', () => {
     renderInRouter('my link', {
       ...defaultProps,