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

Customizable 404 page #493

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

litvinovg
Copy link
Member

@litvinovg litvinovg commented Feb 26, 2025

VIVO GitHub issue
Linked VIVO PR

What does this pull request do?

Specified application of locale selection filter on error requests.
Forward 404 errors to /page_not_found PageController

How should this be tested?

A description of what steps someone could take to:

  • Reproduce the problem in the issue
  • Test new page_not_found error page by opening not existing page, not existing individual

Interested parties

@VIVO-project/vivo-committers @hauschke

Reviewers' expertise

Candidates for reviewing this PR should have some of the following expertises:

  1. Java

Reviewers' report template

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

@hauschke
Copy link
Member

The text in the different languages (pageNotFound.n3) should be decided in English first, then translations can be done or aligned with it.

@chenejac chenejac self-requested a review February 27, 2025 15:42
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.

Customizable 404 page for Vitro/VIVO
2 participants