Skip to content

Conversation

@yunruse
Copy link

@yunruse yunruse commented Sep 19, 2024

fixes:

as much as I agree with other comments that the ?url= segment is a bit awks, IMO it is best practice to use explicit parameters; it is not as if GH Pages lets you do this anyway This kind of long-term shortcut would be best handled on the host level, e.g. with a Cloudflare redirect htmlfil.es/(.+) -> https://html-preview.github.io/?url=$1

fixes:
- html-preview#4

as much as I agree with other comments that the `?url=` segment is a bit awks, IMO it is best practice to use explicit parameters; it is not as if GH Pages lets you do this anyway
This kind of long-term shortcut would be best handled on the host level, e.g. with a Cloudflare redirect `htmlfil.es/(.+) -> https://html-preview.github.io/?url=$1`
@hoijui hoijui merged commit 259d7d4 into html-preview:master Nov 2, 2024
@hoijui
Copy link

hoijui commented Nov 2, 2024

I am sorry for being so late!
Because this is not under my personal account, it seems that I did not get notifications. I should get them now.

Thankl you!

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