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

FIX: Nginx routing for gzip enable #926

Merged
merged 3 commits into from May 4, 2021
Merged

FIX: Nginx routing for gzip enable #926

merged 3 commits into from May 4, 2021

Conversation

adelavega
Copy link
Collaborator

I had to add a couple exception to the routing to enable react-router to work properly

@cypress
Copy link

cypress bot commented May 4, 2021



Test summary

0 7 0 0


Run details

Project Neuroscout
Status Failed
Commit e38c2ba
Started May 4, 2021 11:06 PM
Ended May 4, 2021 11:07 PM
Duration 00:50 💡
OS Linux Ubuntu - 20.04
Browser Electron 87

View run in Cypress Dashboard ➡️


Failures

analysis_builder.spec.js Failed
1 Analysis Builder > analysis builder
home_spec.js Failed
1 The Home Page > successfully loads
2 The Home Page > navigation bar contents
3 The Home Page > splash page contents
4 The Home Page > test sign up button 1
5 The Home Page > test sign up button 2
6 The Home Page > test sign in button

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@codecov-commenter
Copy link

codecov-commenter commented May 4, 2021

Codecov Report

Merging #926 (e38c2ba) into master (5bfecf8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #926   +/-   ##
=======================================
  Coverage   83.16%   83.16%           
=======================================
  Files          62       62           
  Lines        3100     3100           
=======================================
  Hits         2578     2578           
  Misses        522      522           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5bfecf8...e38c2ba. Read the comment docs.

@cypress
Copy link

cypress bot commented May 4, 2021



Test summary

0 7 0 0


Run details

Project Neuroscout
Status Failed
Commit bde96ca ℹ️
Started May 4, 2021 11:06 PM
Ended May 4, 2021 11:07 PM
Duration 00:49 💡
OS Linux Ubuntu - 20.04
Browser Electron 87

View run in Cypress Dashboard ➡️


Failures

analysis_builder.spec.js Failed
1 Analysis Builder > analysis builder
home_spec.js Failed
1 The Home Page > successfully loads
2 The Home Page > navigation bar contents
3 The Home Page > splash page contents
4 The Home Page > test sign up button 1
5 The Home Page > test sign up button 2
6 The Home Page > test sign in button

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@adelavega adelavega merged commit ac7e2d6 into master May 4, 2021
@adelavega adelavega deleted the gzip branch May 4, 2021 23:55
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.

None yet

2 participants