Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement dofollow backlink support #1525

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ebenezerdon
Copy link
Contributor

What does this PR do?

Adds support for selective dofollow backlinks while maintaining secure defaults for external links.

Changes:

  • Modified Link component to support dofollow parameter
  • Maintains noopener security attribute
  • Cleans up URL parameters after processing
  • Keeps nofollow as default for external links

This enables proper backlink implementation for partnership content (e.g., veed.io collaboration) in the updated +page.markdoc file in this PR

Test plan

  • Verify external links default to nofollow
  • Verify ?dofollow=true removes nofollow
  • Verify noopener is maintained
  • Verify URL parameters are cleaned up

- Detect dofollow URL parameter
- Remove nofollow attribute for partnership links
- Clean up URL parameters after processing

This enables proper backlink implementation for partnerships
while maintaining secure defaults
@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Nov 27, 2024

The preview deployment failed. 🔴

Open Build Logs

Last updated at: 2024-11-27 12:49:46 CET

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.

1 participant