Skip to content

Conversation

DipanshKhandelwal
Copy link
Contributor

Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Site has content, commit is valid.

CNAME is not present in gh-pages of https://github.com/DipanshKhandelwal/my-zodiac-sign/tree/gh-pages

@DipanshKhandelwal
Copy link
Contributor Author

@MattIPv4 Oh right thanks, I have added it now.

Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@indus indus added invalid add This PR/issue is regarding adding a new subdomain user This PR/issue is regarding a target on a GitHub user account project This PR has a target of a project on an organisation or user account labels Oct 10, 2018
@indus
Copy link
Member

indus commented Oct 10, 2018

I've already added your requested subdomain to JS.ORGs zonefile. But you have to move the CNAME file to your gh-pages branch to make it work.
I'll merge after we have "contact".CNAME has to be in you gh-pages branch.

@DipanshKhandelwal
Copy link
Contributor Author

@indus Done. I removed it by mistake in the last commit 😅. But now it's fine 👍

@indus indus removed the invalid label Oct 10, 2018
@indus
Copy link
Member

indus commented Oct 10, 2018

Welcome https://zodiacsigns.js.org
(looks like ther is an issue with your asset paths; nevertheless in principle it is working now)

@indus indus merged commit 3c00693 into js-org:master Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add This PR/issue is regarding adding a new subdomain project This PR has a target of a project on an organisation or user account user This PR/issue is regarding a target on a GitHub user account
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants