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.

Sign up for GitHub

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

fix(app): Show 404 pages on more invalid route paths (root and error pages) #3061

Merged

nellh merged 1 commit into master from error-updates

May 21, 2024

Merged

fix(app): Show 404 pages on more invalid route paths (root and error pages) #3061

fix(app): Show 404 pages on more invalid route paths (root and error …

Codecov / codecov/patch failed May 21, 2024 in 1s

0.00% of diff hit (target 42.48%)

View this Pull Request on Codecov

0.00% of diff hit (target 42.48%)

Annotations

Check warning on line 10 in packages/openneuro-app/src/scripts/errors/errorRoute.jsx

codecov / codecov/patch

packages/openneuro-app/src/scripts/errors/errorRoute.jsx#L10

Added line #L10 was not covered by tests

Check warning on line 22 in packages/openneuro-app/src/scripts/errors/errorRoute.jsx

codecov / codecov/patch

packages/openneuro-app/src/scripts/errors/errorRoute.jsx#L12-L22

Added lines #L12 - L22 were not covered by tests

Check warning on line 25 in packages/openneuro-app/src/scripts/errors/errorRoute.jsx

codecov / codecov/patch

packages/openneuro-app/src/scripts/errors/errorRoute.jsx#L24-L25

Added lines #L24 - L25 were not covered by tests

Check warning on line 42 in packages/openneuro-app/src/scripts/routes.tsx

codecov / codecov/patch

packages/openneuro-app/src/scripts/routes.tsx#L42

Added line #L42 was not covered by tests

View more details on Codecov