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 on Github button seems to be broken #3243

Closed
shcheklein opened this issue Feb 1, 2022 · 3 comments
Closed

Edit on Github button seems to be broken #3243

shcheklein opened this issue Feb 1, 2022 · 3 comments
Assignees
Labels
A: website Area: website p0-critical Affects users in a bad way at the moment

Comments

@shcheklein
Copy link
Member

I detected this trough the Hexometer monitoring:

Screen Shot 2022-01-31 at 6 26 34 PM

Let's do a postmortem and analyze how could we improve this. Seems like the right way is to pay more attention to the link checker ? we need to see if it was able to detect this in the first place.

@shcheklein shcheklein added p0-critical Affects users in a bad way at the moment A: website Area: website labels Feb 1, 2022
@shcheklein
Copy link
Member Author

Screen Shot 2022-01-31 at 6 28 52 PM

@julieg18 julieg18 self-assigned this Feb 1, 2022
@rogermparent
Copy link
Contributor

Thanks @julieg18 for the fix! This was wholly on me because I made a typo in moving the logic for generating GitHub links: I set the repo as iterative/dvc instead of iterative/dvc.org

Seems like the right way is to pay more attention to the link checker ?

The PR link checker wouldn't have caught this, as it only inspects sources, and I'm pretty sure the whole-site checker does as well. The resulting links from these "edit on GitHub" buttons are only ever put together dynamically at the end of the process with a template string, so the link checker never reads them.

@rogermparent
Copy link
Contributor

Just merged the fix PR and made sure the buttons work, and they do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: website Area: website p0-critical Affects users in a bad way at the moment
Projects
None yet
Development

No branches or pull requests

3 participants