fix: handle optional pathname in StyledLink component

CodeForAfrica · Feb 4, 2025 · d003155 · d003155
1 parent e1171fd
commit d003155
Showing 1 changed file with 2 additions and 2 deletions.
diff --git 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,