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

Make error pages translatable #3405

Merged
merged 2 commits into from
May 29, 2020
Merged

Make error pages translatable #3405

merged 2 commits into from
May 29, 2020

Conversation

sephrat
Copy link
Contributor

@sephrat sephrat commented May 27, 2020

I could not find a way to test 500 and 503 error pages.

@JC5
Copy link
Member

JC5 commented May 27, 2020

Browse to /error to test those. I’ll review later :)

@sephrat
Copy link
Contributor Author

sephrat commented May 27, 2020

Browse to /error to test those.

/error does show FireflyException.twig, but how do I test 500 and 503 ? I tried /error/500 but this gives a 404.

@JC5
Copy link
Member

JC5 commented May 28, 2020

Mmmm. I'm just not sure of uncaught errors (which would cause the 500).

The 503 can be tested, I just realised, by turning on maintenance mode. php artisan maintenance on I think it is.

@sephrat
Copy link
Contributor Author

sephrat commented May 28, 2020

Right, I managed to test 503 with php artisan down. Thanks.

Will try to break things to get that 500 error page. Fun fact: if you throw an Exception in FireflyException.twig you get an infinite loop. :-)
Edit: OK I just found a way: route to 500 instead of FireflyException in /app/Exceptions/Handler.php.

Tests OK!

@JC5
Copy link
Member

JC5 commented May 28, 2020

Nice! I've written down the infinite loop. Just to see if I can catch it somehow.

@JC5 JC5 merged commit ead2cf9 into firefly-iii:develop May 29, 2020
@JC5
Copy link
Member

JC5 commented May 29, 2020

Merged 👍

@sonarcloud
Copy link

sonarcloud bot commented May 29, 2020

SonarCloud Quality Gate failed.

Bug B 1 Bug
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 19 Code Smells

No Coverage information No Coverage information
5.2% 5.2% Duplication

@sephrat sephrat deleted the error_pages_translation branch June 4, 2020 10:41
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants