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

dot.js.org #3597

Closed
wants to merge 2 commits into from
Closed

dot.js.org #3597

wants to merge 2 commits into from

Conversation

epoberezkin
Copy link
Contributor

@MattIPv4
Copy link
Member

Any reason this is targeting your fork and not http://olado.github.io/doT ?

@MattIPv4 MattIPv4 changed the title add dot.js.org dot.js.org Dec 14, 2019
@MattIPv4 MattIPv4 added the awaiting response This PR is waiting for a response from the requester label Dec 14, 2019
@epoberezkin
Copy link
Contributor Author

@MattIPv4 I've requested that olado/dot settings are updated - I don't have access to settings there: olado/doT#236 (comment)

http://olado.github.io/doT/ is created from another repo that I don't maintain.

I've created the fork to prepare and test the site before it is merged to the main repo.

But actually you are right, until the site is moved to the main repo we can just point to the existing site (although it probably won't work because I can't commit CNAME there?).

Anyway, we don't need to merge it right now.

@indus
Copy link
Member

indus commented Dec 14, 2019

I would prefer to add the subdomain when we can point it to the main branch and not the fork.

@indus
Copy link
Member

indus commented Dec 21, 2019

@epoberezkin any progress?

@epoberezkin
Copy link
Contributor Author

didn't hear from @olado yet, sorry

@indus
Copy link
Member

indus commented Dec 29, 2019

@epoberezkin Would you mind to close this PR and open a new one when you got feedback from the repo owner?

@indus indus closed this Jan 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting response This PR is waiting for a response from the requester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants