"Edit page" link on TOC is broken

[visvirial]

"Edit page" link on TOC is broken

[wackerow]

A patch was put up for this, but it appears there is still a bug with it:

// Inside /src/lib/utils/editPath.ts, we return:

join(EDIT_CONTENT_URL, CONTENT_DIR, relativePath, "index.md")

EDIT_CONTENT_URL has https:// at the beginning of it... we cannot pass this to join() because that function will collapse repeating slashes (/) down to only one, as it's forming a path.

We should use construct a URL with new URL(path, base) instead and return the href value from the resulting object, passing join(CONTENT_DIR, relativePath, "index.md") as the path argument, and EDIT_CONTENT_URL as the base... this will properly parse these into a URL.

ie. (new URL(join(CONTENT_DIR, relativePath, "index.md"), EDIT_CONTENT_URL)).href

@HiroyukiNaito Assigning you for this =)

[tutizaraz]

It seems that this has already been fixed but not deployed to production

[HiroyukiNaito]

If it is not fixed, I would like to work on the issue.

[HiroyukiNaito]

@wackerow
Is it fixed? If not, I doubt it is caused only by SSG. Is there any way to build same as ethereum.org SSG for locating the cause?

[HiroyukiNaito]

@wackerow Thanks for the tips! I work on it.

[wackerow]

Hey @HiroyukiNaito! Thanks for offering to patch this... Just want to mention that this bug is somewhat important since it's breaking all the edit links in the docs. We were hoping to get this patched ASAP...

Any chance you're able to make this patch in the next few hours? Otherwise I may put up a patch so we can hot-fix the deploy planned for later today. Sorry for the time crunch 🙏

[wackerow]

Hey @HiroyukiNaito, apologies for jumping in on this, but given the nature of this bug we wanted to get a patch up for this ASAP... I put up a PR to fix, but thank you for offering to help!

[HiroyukiNaito]

@wackerow Sure, thanks for notifying that!