Skip to content

Feature: resolve relative links#747

Open
daniel-simpson wants to merge 5 commits intobastienwirtz:mainfrom
daniel-simpson:feature/relative-links
Open

Feature: resolve relative links#747
daniel-simpson wants to merge 5 commits intobastienwirtz:mainfrom
daniel-simpson:feature/relative-links

Conversation

@daniel-simpson
Copy link
Copy Markdown
Contributor

Description

Implementation for feature request: #746

No dependencies other than the window.location APIs provided by the browser.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I've read & comply with the contributing guidelines
  • I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
  • I have made corresponding changes to the documentation (README.md).
  • I've checked my modifications for any breaking changes, especially in the config.yml file

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 15, 2024

Deploy Preview for homer-demo-content ready!

Name Link
🔨 Latest commit d5d3e00
🔍 Latest deploy log https://app.netlify.com/sites/homer-demo-content/deploys/65e54f3a09cb200008ea93df
😎 Deploy Preview https://deploy-preview-747--homer-demo-content.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@saghul
Copy link
Copy Markdown

saghul commented Mar 19, 2024

This would be very useful for me as well 😅

Any chance of getting this merged?

@bastienwirtz bastienwirtz force-pushed the main branch 2 times, most recently from 12cad12 to 474dc3a Compare February 23, 2025 13:28
@caesay
Copy link
Copy Markdown

caesay commented Mar 10, 2025

This is awesome (and I hope it gets merged) but it's not quite complete. The same problem applies to service/integrations, and some of those use an endpoint option instead of / with the url option.

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.

3 participants