pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/processing/processing-website/pull/515

s-be6017ec12798e73.css" /> added home link on 404 page by praveen-rikhari · Pull Request #515 · processing/processing-website · GitHub
Skip to content

added home link on 404 page - #515

Open
praveen-rikhari wants to merge 1 commit into
processing:mainfrom
praveen-rikhari:praveen/fix-404-page-with-home-link
Open

added home link on 404 page#515
praveen-rikhari wants to merge 1 commit into
processing:mainfrom
praveen-rikhari:praveen/fix-404-page-with-home-link

Conversation

@praveen-rikhari

@praveen-rikhari praveen-rikhari commented Feb 20, 2024

Copy link
Copy Markdown

Description

Now you can see back to home page link on the 404 page

@netlify

netlify Bot commented Feb 20, 2024

Copy link
Copy Markdown

Deploy Preview for processing-faf822 ready!

Name Link
🔨 Latest commit f5e475a
🔍 Latest deploy log https://app.netlify.com/sites/processing-faf822/deploys/65d4d2ff81179e000891cd14
😎 Deploy Preview https://deploy-preview-515--processing-faf822.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@praveen-rikhari
praveen-rikhari force-pushed the praveen/fix-404-page-with-home-link branch from 2501969 to f5e475a Compare February 20, 2024 16:27
@SableRaf
SableRaf requested a review from Stefterv July 12, 2024 11:49
@SableRaf

Copy link
Copy Markdown
Collaborator

@Stefterv Since you were recently looking at the 404 page maybe you'll have some thoughts about this one.

@Stefterv Stefterv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good apart from the unnecessarily added package
The language/content probably needs some work

Comment thread i18n/react-intl/es.json

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a Spanish translation

Comment thread package.json

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unnecessary ?

Comment thread i18n/react-intl/en.json
"inUse": "In use",
"notFound": "NOT FOUND",
"notFoundText": "You just hit a route that doesn't exist... the sadness.",
"backToHome": "back to home",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitalisation?

@SableRaf

Copy link
Copy Markdown
Collaborator

@praveen-rikhari If you're still interested in this PR, could you take a look at the comments above? Thank you for your contribution!

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.

3 participants

pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy