Skip to content

Conversation

@persd
Copy link
Contributor

@persd persd commented Jan 14, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds domain whitelisting functionality for content links, allowing configuration of SEO-friendly link attributes based on whether domains are whitelisted. The feature enables automatic addition of nofollow and noindex attributes to non-whitelisted external links in paragraph content, while allowing custom rel attributes for whitelisted domains.

Changes:

  • New SeoLinkHelper module to process HTML links and apply rel attributes based on domain whitelist configuration
  • Configuration schema additions in SEOWebsitesConfig for domain whitelist settings
  • Integration of link processing in ParagraphBlock component after link replacement
  • Version bump from 4.4.3 to 4.5.0 (appropriate for a minor feature addition)

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
src/helpers/SeoLinkHelper.ts New helper module that processes HTML anchor tags, checks domains against whitelist, and applies appropriate rel attributes
src/helpers/ConfigHelper.ts Added ConfigHelper_getContentLinksConfig function to retrieve content links configuration
src/configs/SEOWebsitesConfig.ts Added configuration schema for contentLinks including domain whitelist and rel attribute settings
src/components/widgets/Story/StoryContent/StoryContentBlocks/ParagraphBlock.astro Integrated SeoLinkHelper processing after LinkReplacerHelper
package.json Version bumped to 4.5.0 for minor feature addition

@persd persd changed the title add whitelisted domains for content links add a whitelist of domains for links in content Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants