Skip to content

[1.x] Fix reference URL not matching case sensitively#1422

Merged
ksen0 merged 2 commits into
processing:mainfrom
limzykenneth:main-deps-update
Jun 1, 2026
Merged

[1.x] Fix reference URL not matching case sensitively#1422
ksen0 merged 2 commits into
processing:mainfrom
limzykenneth:main-deps-update

Conversation

@limzykenneth
Copy link
Copy Markdown
Member

Counterpart to #1421

@limzykenneth
Copy link
Copy Markdown
Member Author

@ksen0
Copy link
Copy Markdown
Member

ksen0 commented Jun 1, 2026

Documenting discord testing chat:

@nbogie :

The above is working on the PR preview for beta but not the preview for main.
E.g. the following won't work all lower-case:
https://92b3ff0c-p5-js-website.p5js-org-account.workers.dev/reference/p5/createvector/
https://92b3ff0c-p5-js-website.p5js-org-account.workers.dev/reference/p5.vector/fromangle/
https://92b3ff0c-p5-js-website.p5js-org-account.workers.dev/reference/p5.vector/fromangles/
https://92b3ff0c-p5-js-website.p5js-org-account.workers.dev/reference/p5/buildgeometry/
The equivalents URLs all work on the other preview (the one for beta).

@limzykenneth :

Yes that's expected, I haven't yet leave a comment in there mentioning it. I haven't dig into detail why that is the case, it may be some patch that wasn't applied to the main branch, but we are planning a 1.x > 2.x switch for the website soon which will center 2.x more so this is not a priority at the moment

@ksen0 ksen0 merged commit 7fdab27 into processing:main Jun 1, 2026
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants