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

Ga4 with helmet #1105

Merged
merged 12 commits into from Jul 1, 2023
Merged

Ga4 with helmet #1105

merged 12 commits into from Jul 1, 2023

Conversation

rwblair
Copy link
Collaborator

@rwblair rwblair commented Jun 16, 2023

I can't get the docker-compose to build on deepdream so this still needs to be tested there.

@rwblair
Copy link
Collaborator Author

rwblair commented Jun 16, 2023

Don't actually need gpu support, currently nginx and flask containers aren't talking on deepdream.

@codecov-commenter
Copy link

Codecov Report

Merging #1105 (e0e7bf2) into master (a3e7f32) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master    #1105   +/-   ##
=======================================
  Coverage   76.69%   76.69%           
=======================================
  Files          64       64           
  Lines        3334     3334           
=======================================
  Hits         2557     2557           
  Misses        777      777           

@cypress
Copy link

cypress bot commented Jul 1, 2023

1 flaky tests on run #714 ↗︎

0 8 0 0 Flakiness 1

Details:

Merge e0e7bf2 into a3e7f32...
Project: Neuroscout Commit: 2599fbfac5 ℹ️
Status: Passed Duration: 01:43 💡
Started: Jul 1, 2023 10:17 PM Ended: Jul 1, 2023 10:19 PM
Flakiness  cypress/e2e/analysis_builder.spec.js • 1 flaky test

View Output Video

Test Artifacts
Analysis Builder > analysis builder Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@adelavega adelavega merged commit e09ef00 into master Jul 1, 2023
4 of 5 checks passed
@adelavega adelavega deleted the ga4_with_helmet branch July 1, 2023 22:32
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

3 participants