-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
fix: docker nginx 404 error #751
Conversation
👷 Deploy request for geoguess2 pending review.Visit the deploys page to approve it
|
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #751 +/- ##
=======================================
Coverage 52.12% 52.12%
=======================================
Files 53 53
Lines 752 752
Branches 209 209
=======================================
Hits 392 392
Misses 328 328
Partials 32 32 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look good, thanks 👍
@all-contributors please add @HelloJowet for code |
I've put up a pull request to add @HelloJowet! 🎉 |
1 failed test on run #713 ↗︎
Details:
tests/e2e/specs/game/gameSingleCountry.test.js • 1 failed test • chrome
Review all test suite changes for PR #751 ↗︎ |
Types of Changes
Description
If I run the application with docker and try to access subdirectories like
/history
, nginx returns a 404 error. I added a nginx configuration file to prevent this issue.Close #619
How Has This Been Tested?
After my changes I can now access subdirectories like
/history
directly without any errors.