From d0031557d5465e83d23298c966e44494c1cb9724 Mon Sep 17 00:00:00 2001 From: Kevin Koech Date: Tue, 4 Feb 2025 11:14:53 +0300 Subject: [PATCH] fix: handle optional pathname in StyledLink component --- packages/commons-ui-next/src/Link/StyledLink.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/commons-ui-next/src/Link/StyledLink.js b/packages/commons-ui-next/src/Link/StyledLink.js index 956af28ea..cef02bf09 100644 --- a/packages/commons-ui-next/src/Link/StyledLink.js +++ b/packages/commons-ui-next/src/Link/StyledLink.js @@ -74,7 +74,7 @@ const StyledLink = React.forwardRef(function Link(props, ref) { } = props; // https://nextjs.org/docs/app/api-reference/components/link#href-required - const url = typeof href === "string" ? href : href.pathname; + const url = typeof href === "string" ? href : href?.pathname; const isExternal = isExternalUrl(url); if (isExternal) { const externalLinkProps = { @@ -117,7 +117,7 @@ const StyledLink = React.forwardRef(function Link(props, ref) { StyledLink.propTypes = { as: PropTypes.oneOfType([PropTypes.object, PropTypes.string]), className: PropTypes.string, - href: PropTypes.string, + href: PropTypes.string.isRequired, legacyBehavior: PropTypes.bool, linkAs: PropTypes.oneOfType([PropTypes.object, PropTypes.string]), locale: PropTypes.string,