Skip to content
This repository was archived by the owner on Sep 1, 2022. It is now read-only.

Update URLs to be correct on GHES#22

Merged
crichID merged 3 commits intomasterfrom
correct-links
May 9, 2019
Merged

Update URLs to be correct on GHES#22
crichID merged 3 commits intomasterfrom
correct-links

Conversation

@brianamarie
Copy link
Copy Markdown
Contributor

This PR updates the URLs for several responses so they will also work on GHES.

cc @JasonEtco as this uses the new {{ repoUrl }} variable.
cc @githubtraining/learning-engineering, @githubtraining/trainers

@brianamarie brianamarie self-assigned this May 8, 2019
@brianamarie brianamarie added the bug Something isn't working label May 8, 2019
@brianamarie
Copy link
Copy Markdown
Contributor Author

@hectorsector @JasonEtco Do you know how I can reference the URL to the pages site correctly? I've commented by the line that needs changed, but I'm not sure the right way to do it.

@brianamarie
Copy link
Copy Markdown
Contributor Author

This is ready for review! I'd like to 🚢 with 1 👍 but will consider merging without review to make sure this can be on ghe tomorrow. cc @githubtraining/learning-engineering @githubtraining/trainers for 👀

Copy link
Copy Markdown
Contributor

@crichID crichID left a comment

Choose a reason for hiding this comment

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

@brianamarie I tested on .com and it looks good! I'm excited that it even applies my cname 😉

@crichID
Copy link
Copy Markdown
Contributor

crichID commented May 9, 2019

I'm going to go ahead and merge this in under the assumption that @brianamarie has already 🍰

@crichID crichID merged commit 0773727 into master May 9, 2019
@crichID crichID deleted the correct-links branch May 9, 2019 16:49
crichID added a commit that referenced this pull request Feb 7, 2020
Update URLs to be correct on GHES
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants