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

add relative url to all resource urls #60

Merged
merged 14 commits into from
Sep 19, 2024

Conversation

mingness
Copy link
Member

@mingness mingness commented Sep 18, 2024

What this PR does is change all URLs in the website to relative.

This is needed so the new automatic Pull Request "Previewer" produces a temporary website in which links and media refer to that preview website (instead of to the live website).

Otherwise clicked links take the user to the live website, and new images are not found (because they do not yet exist in the live website).

@mingness mingness linked an issue Sep 18, 2024 that may be closed by this pull request
@mingness mingness force-pushed the 56-fix-pr-preview-to-work-with-jekyll branch from d6b666b to b0b96c7 Compare September 18, 2024 17:42
Copy link

github-actions bot commented Sep 18, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-09-19 17:23 UTC

@mingness
Copy link
Member Author

@hamoid I'll need your help to edit url to open call. I don't see how you added the image and link.

Current unresolved issues - bg image on homepage not resolving, also video on art market page not playing.

@mingness
Copy link
Member Author

@hamoid could I pass the torch to you? This isn't an urgent issue either. So leave it if you'd like. I'll just need to take a break for a couple weeks to prioritize other work.

@hamoid
Copy link
Member

hamoid commented Sep 18, 2024

Sure. Thanks for doing this already!! I'll take a look.

@hamoid hamoid self-requested a review September 18, 2024 20:26
hamoid
hamoid previously approved these changes Sep 18, 2024
@mingness mingness changed the title add relative url to all resource urls [WIP] add relative url to all resource urls Sep 18, 2024
@mingness mingness merged commit 1552327 into main Sep 19, 2024
1 check passed
@mingness mingness deleted the 56-fix-pr-preview-to-work-with-jekyll branch September 19, 2024 17:23
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.

fix PR preview to work with jekyll
3 participants