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

Edit Redirects to 404 path #228

Open
rinoldsimon opened this issue Apr 21, 2021 · 6 comments
Open

Edit Redirects to 404 path #228

rinoldsimon opened this issue Apr 21, 2021 · 6 comments

Comments

@rinoldsimon
Copy link
Contributor

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

@ijlee2
Copy link
Member

ijlee2 commented Apr 21, 2021

@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:

@jenweber
Copy link
Contributor

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 ?

@ijlee2
Copy link
Member

ijlee2 commented Apr 21, 2021

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

@jenweber
Copy link
Contributor

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
Copy link
Contributor

Step one is done and the links work again.

@Windvis
Copy link
Contributor

Windvis commented Oct 30, 2021

This commit seems to add the needed configuration: 2b376bc

So I think this issue can be closed?

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

No branches or pull requests

4 participants