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

Add nice 404 page #45

Open
Tom-van-Woudenberg opened this issue Jun 17, 2024 · 4 comments
Open

Add nice 404 page #45

Tom-van-Woudenberg opened this issue Jun 17, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@Tom-van-Woudenberg
Copy link
Member

To be compatible with: #39
@severinbratus, am I correct that the current 404 page is described here:

document.body.innerHTML = "<h1>Page Not Found</h1>";
and that the current 404.html and 404.md can be deleted?
Is it possible to use this default 404 page only if no custom 404 page is provided on the root of the repo to be in line with the github documentation?

@severinbratus
Copy link
Contributor

severinbratus commented Jun 18, 2024

Yes, that is correct. Yes, this should be possible. I see two tasks to do here:

  • Make a better default 404 page (e.g. also with a list of deployed branches, or a link to the latest successful summary)
  • Use the 404.html in the root dir if provided

@AleksSliw
Copy link
Collaborator

I am editing the template for the generated 404 page and a question popped up. What should be the expected behaviour for 404 redirect? inputting a wrong URL in the form of valid-repository/wrong-name will redirect to valid-repository/main/wrong-name (which is still invalid and will show 404), while inputting valid-repository/right-branch/wrong-name will just show the 404 page with that same URL.

Im assuming the idea is to redirect to the main branch if the repository exists?

@Tom-van-Woudenberg
Copy link
Member Author

I think the 404 page should never redirect, only show a button to bring you to the root of valid-repository
There'll always be a page on root.

@Tom-van-Woudenberg
Copy link
Member Author

Maybe we can make the 404 page from the new WordPress website? In that theme

@Tom-van-Woudenberg Tom-van-Woudenberg added the enhancement New feature or request label Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

3 participants