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

Remove CNAME from GitHub Pages. #364

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

rupl
Copy link

@rupl rupl commented May 20, 2018

Howdy, I was looking into using your plugin and got redirected to some spam from clicking the homepage link on NPM. At first glance, it seems like it will go to GitHub Pages but due to the CNAME it redirects to http://smoothstate.com which is currently forwarding people to spam ☹️

Meanwhile we can't see the GitHub Pages URL because of the CNAME file. This PR fixes the problem and restores docs on the github.io domain. GitHub Pages now offers free HTTPS so it's a fine solution, but if you do want a "prettier" URL maybe you could grab a namespace on https://dns.js.org — it's free for an open-source project like yours.

@rupl
Copy link
Author

rupl commented May 20, 2018

Because I forked your repo to alter the gh-pages branch you can see my functioning copy of your docs once the CNAME is removed: https://rupl.github.io/smoothState.js/

@rupl rupl changed the title Remove CNAME Remove CNAME from GitHub Pages May 21, 2018
@rupl rupl changed the title Remove CNAME from GitHub Pages Remove CNAME from GitHub Pages. Fixes #346 May 22, 2018
@rupl
Copy link
Author

rupl commented May 22, 2018

Fixes #346. Fixes #342. Fixes #340.

@rupl rupl changed the title Remove CNAME from GitHub Pages. Fixes #346 Remove CNAME from GitHub Pages. May 22, 2018
@rupl
Copy link
Author

rupl commented Jul 23, 2019

Fixes #373.

@leapooo
Copy link

leapooo commented Oct 11, 2021

here is a great bongs headshop

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