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

feat: redirect not found page to homepage #8

Merged
merged 3 commits into from
Sep 17, 2022

Conversation

takanome-dev
Copy link
Contributor

@takanome-dev takanome-dev commented Sep 16, 2022

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Description

Add a 404 page that redirects to the homepage

Related Tickets & Documents

Fixes #3

Mobile & Desktop Screenshots/Recordings

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@netlify
Copy link

netlify bot commented Sep 16, 2022

❌ Deploy Preview for shimmering-puppy-475abf failed.

Name Link
πŸ”¨ Latest commit de6493f
πŸ” Latest deploy log https://app.netlify.com/sites/shimmering-puppy-475abf/deploys/6324f72b5e7c22000867c2b4

@netlify
Copy link

netlify bot commented Sep 16, 2022

βœ… Deploy Preview for contributor-info ready!

Name Link
πŸ”¨ Latest commit 4d1ebba
πŸ” Latest deploy log https://app.netlify.com/sites/contributor-info/deploys/6325887d7221b800080dfe2a
😎 Deploy Preview https://deploy-preview-8--contributor-info.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 settings.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks Passed!

@0-vortex
Copy link
Contributor

@takanome-dev https://beta.contributor.info is up now, you should be able to merge with beta and get a deploy preview going! πŸ•

@takanome-dev
Copy link
Contributor Author

@0-vortex done βœ”οΈ

@0-vortex 0-vortex merged commit 9da5e11 into open-sauced:beta Sep 17, 2022
Copy link
Contributor

@0-vortex 0-vortex left a comment

Choose a reason for hiding this comment

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

LTGM! πŸ‘

github-actions bot pushed a commit that referenced this pull request Sep 17, 2022
### [1.0.1-beta.1](v1.0.0...v1.0.1-beta.1) (2022-09-17)

### πŸ§‘β€πŸ’» Code Refactoring

* improve redirect not found page and route ([#8](#8)) ([9da5e11](9da5e11)), closes [#3](#3)
@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.0.1-beta.1 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

@takanome-dev takanome-dev deleted the redirect-404-to-home branch September 17, 2022 21:22
github-actions bot pushed a commit that referenced this pull request Sep 27, 2022
## [1.1.0](v1.0.0...v1.1.0) (2022-09-27)

### πŸ§‘β€πŸ’» Code Refactoring

* improve redirect not found page and route ([#8](#8)) ([9da5e11](9da5e11)), closes [#3](#3)

### πŸ• Features

* Adds working search to home page ([#11](#11)) ([f88bf74](f88bf74)), closes [#4](#4)
@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.1.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: force redirects for not found 404 to "/"
2 participants