Skip to content

Respect host and protocol options in vite_preload_tag#413

Merged
ElMassimo merged 2 commits intoElMassimo:mainfrom
rywall:patch-1
Nov 5, 2024
Merged

Respect host and protocol options in vite_preload_tag#413
ElMassimo merged 2 commits intoElMassimo:mainfrom
rywall:patch-1

Conversation

@rywall
Copy link
Copy Markdown
Contributor

@rywall rywall commented Oct 23, 2023

Description 📖

When specifying the host and protocol options for a vite_javascript_tag, we want the preload tags to also respect those options.

Background 📜

This was causing an issue where the preload tags would not use the CDN specified for the vite_javascript_tag.

When specifying the host and protocol options for a vite_javascript_tag we want the preload tags to also respect those options.
@ElMassimo ElMassimo force-pushed the main branch 2 times, most recently from d55cc92 to ebe50b8 Compare July 17, 2024 14:43
@sleistner
Copy link
Copy Markdown

Hi @ElMassimo,
Just wanted to kindly follow up on this PR. It’s been a while (about a year now), and we’d appreciate your review when you have a moment. Please let me know if any changes or further discussions are needed to move it forward.
Thank you for your time!

@ElMassimo ElMassimo merged commit bcd8151 into ElMassimo:main Nov 5, 2024
@sleistner
Copy link
Copy Markdown

Awesome! 👍🏻

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