cli-guides icon indicating copy to clipboard operation
cli-guides copied to clipboard

Edit Redirects to 404 path

Open rinoldsimon opened this issue 4 years ago • 6 comments

Editing guides at all the pages are redirecting to 404

When I click the edit icon at the top right corner https://cli.emberjs.com/release/appendix/dev-tools/, It takes me to https://github.com/ember-learn/cli-guides/edit/master/guides/appendix/dev-tools.md (404)

Actually, it should take me to https://github.com/ember-learn/cli-guides/edit/main/guides/appendix/dev-tools.md

rinoldsimon avatar Apr 21 '21 09:04 rinoldsimon

@rinoldsimon Thank you for letting us know of this issue. We recently updated the default branch name for cli-guides to main, and didn't realize the scenario with the edit link.

The edit link is hardcoded in guidemaker-ember-template. Possible solutions are:

ijlee2 avatar Apr 21 '21 11:04 ijlee2

I think we can add this to Guidemaker as an addon config pretty easily. I’ll try to look at it today or ask Chris for help when we pair on Thursday.

A third option is to recreate the “master” branch so that the links go somewhere. I think I will do that as well, while we work on the other task. How does that sound, @ijlee2 ?

jenweber avatar Apr 21 '21 12:04 jenweber

@jenweber Yep, a temporary revert of the branch name is okay with me. Thanks for your help!

ijlee2 avatar Apr 21 '21 12:04 ijlee2

We don’t have to revert anything, thankfully. I will branch from main, call the new branch master, and leave main as the default. I’ll do that now. I’ll also make sure that removing it is tracked in an issue.

jenweber avatar Apr 21 '21 13:04 jenweber

Step one is done and the links work again.

jenweber avatar Apr 21 '21 13:04 jenweber

This commit seems to add the needed configuration: https://github.com/ember-learn/cli-guides/commit/2b376bcd9e3d9d1eae4eb3a669221faed9a16017

So I think this issue can be closed?

Windvis avatar Oct 30 '21 12:10 Windvis