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

broken links for p5js.org/get-started at hello.p5js.org #479

Open
villares opened this issue Aug 7, 2024 · 5 comments
Open

broken links for p5js.org/get-started at hello.p5js.org #479

villares opened this issue Aug 7, 2024 · 5 comments
Labels
Bug: Broken Link Bug Something isn't working

Comments

@villares
Copy link

villares commented Aug 7, 2024

Most appropriate sections of the p5.js website?

Other - GitHub repository Read-me and hello.p5js.org page.

Actual Behavior

The https://p5js.org/get-started/ page is missing, and it is linked to at some places.
I first found about it at the end of the interactive https://hello.p5js.org presentation, then at the main repo https://github.com/processing/p5.js (update: the read-me was fixed):

There should be a get-started page, maybe it was renamed or removed?

If it was renamed, redirecting from the old link could be a reasonable fix. If it was removed, a placeholder page linking to replacement resources might be a good idea.

Expected Behavior

Getting to a "Get Started" page.

update: it should perhaps get you to https://p5js.org/tutorials/get-started/

Steps to reproduce

Would you like to work on the issue?

yes, but wouldn't know where to start

@villares villares added the Bug Something isn't working label Aug 7, 2024
@Dharmendra016
Copy link

@villares ,I think you can solve the docs issue by replacing the existing link in the README.md file with the correct one.
and the README.md file is in p5.js repo.

@villares
Copy link
Author

villares commented Aug 8, 2024

the link embeded in there presentation is harder to replace, so a redirect would be nice too...

@flors
Copy link

flors commented Sep 5, 2024

Hi @villares @Dharmendra016 it seems there is a duplicated issue in the p5js repository: processing/p5.js#7208
And there is a recent PR to solve that issue. Hope it helps!

@villares
Copy link
Author

villares commented Sep 5, 2024

this is great, but there is still the redirecion or other way of solving the broken link inside hello.p5js.org

@villares villares changed the title broken links for p5js.org/get-started at hello.p5js.org and github.com/processing/p5.js broken links for p5js.org/get-started at hello.p5js.org Sep 5, 2024
@ibrand
Copy link

ibrand commented Sep 6, 2024

@villares I did a search through this codebase and also perused the Processing github org and I can't find the code for the video. I'm a new contributor (I've written that initial issue) so I might be missing something. A redirect is a good idea because it's durable, but if anyone knows where the video code lives, the change needed would be to replace the link right here:

      <a class="btn btn-primary" target="_blank" href="http://p5js.org/get-started/">
       <i class="fa fa-mortar-board fa-2x"></i><br>
       Get Started
      </a>  

that's nested in the <div id="cta"> section with the link https://p5js.org/tutorials/get-started/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Broken Link Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants