NotificationsYou must be signed in to change notification settings
Fork195
Star58

prevent double-slash in asset paths (//assets/ --> /assets/)#925

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

perminder-17 wants to merge4 commits intoprocessing:2.0

base:2.0

Choose a base branch

fromperminder-17:extra-slash

Open

prevent double-slash in asset paths (//assets/ --> /assets/)#925

perminder-17 wants to merge4 commits intoprocessing:2.0fromperminder-17:extra-slash

Conversation

Copy link

Contributor

perminder-17 commentedAug 2, 2025

Resolves#912

I also build the reference again in the website, and commited the regenerated MDX, and included those files in the PR as well.

fixing-extra-slash

19eb58d

perminder-17 requested review fromdavepagurek andksen0

August 2, 2025 14:09

perminder-17 linked an issue

Aug 3, 2025

that may beclosed by this pull request

Typo in Example Code - textToContours()#912

Open

Copy link

Member

ksen0 commentedAug 5, 2025

Hi@perminder-17 , thanks for this, it looks good. OK to merge!

Not sure if you've come across it, but FYI there's a related, but not quite duplicate (onmain not2.0, and about absolute not relative links), is#878 (see also thecomment on that PR)

ksen0 approved these changes

Aug 5, 2025

View reviewed changes

src/scripts/builders/reference.tsShow resolvedHide resolved

Copy link

ContributorAuthor

perminder-17 commentedAug 11, 2025

Not sure if you've come across it, but FYI there's a related, but not quite duplicate (on main not 2.0, and about absolute not relative links), is#878 (see also the#892 (comment))

Just added a comment on that,#892 (comment)
Looks like I found the issue contributor was facing. I can probably help her out :)