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

Fix broken links in contributor_docs/documentation_style_guide.md #7220

Open
1 of 17 tasks
aleannab opened this issue Sep 4, 2024 · 3 comments · May be fixed by #7359
Open
1 of 17 tasks

Fix broken links in contributor_docs/documentation_style_guide.md #7220

aleannab opened this issue Sep 4, 2024 · 3 comments · May be fixed by #7359
Assignees
Labels

Comments

@aleannab
Copy link
Contributor

aleannab commented Sep 4, 2024

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

v1.10.0 (latest)

Web browser and version

Firefox v130.0, though would affect all.

Operating system

Windows, though would affect all.

Steps to reproduce this

Steps:

  1. Go to: https://github.com/processing/p5.js/blob/main/contributor_docs/documentation_style_guide.md
  2. Click on respective links (listed in table below).
  3. Get a 404 error.

Solutions:

Replace listed broken links to the correct links. I did look through the p5.js-website repo, and couldn't find matching file paths.

Source link text: Current link: Suggested link:
src/data/examples/assets https://github.com/processing/p5.js-website/tree/main/src/data/examples ???
src/templates/pages/reference/assets https://github.com/processing/p5.js-website/tree/main/src/templates/pages/reference/assets ???
src/assets/learn https://github.com/processing/p5.js-website/tree/main/src/assets/learn ???
@aleannab aleannab added the Bug label Sep 4, 2024
@iamniky
Copy link

iamniky commented Sep 6, 2024

I'd like to work on this issue

@iamnikhilranjan
Copy link

iamnikhilranjan commented Oct 18, 2024

Hi @aleannab, I’m interested in working on this issue. Could you please assign it to me ?

@iamnikhilranjan
Copy link

Thank you @perminder-17 ! I've opened a pull request within p5.js library repo

Here is the [PR Link] (#7359)

@iamnikhilranjan iamnikhilranjan linked a pull request Nov 4, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants