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

Merged
merged 10 commits into from
Mar 31, 2025
Merged

Customizable 404 page #493

merged 10 commits into from
Mar 31, 2025

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
Copy link
Contributor

@chenejac chenejac left a comment

Choose a reason for hiding this comment

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

@litvinovg do we need this line anymore? Do we need this, this and this file anymore?

@litvinovg
Copy link
Member Author

@chenejac I removed not used error templates and not used translations.

chenejac
chenejac previously approved these changes Mar 8, 2025
Copy link
Contributor

@chenejac chenejac left a comment

Choose a reason for hiding this comment

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

@litvinovg thanks, looks good and works as expected

@chenejac chenejac requested review from wwtamu and gneissone and removed request for gneissone March 8, 2025 12:29
@chenejac chenejac merged commit b427cb7 into vivo-project:main Mar 31, 2025
1 check passed
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
5 participants