-
Notifications
You must be signed in to change notification settings - Fork 40
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
front: add route e2e test for operational studies #8042
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #8042 +/- ##
============================================
+ Coverage 28.13% 28.25% +0.12%
- Complexity 2120 2123 +3
============================================
Files 1302 1300 -2
Lines 158914 158376 -538
Branches 3179 3187 +8
============================================
+ Hits 44715 44757 +42
+ Misses 112293 111707 -586
- Partials 1906 1912 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
f09eaea
to
bf26954
Compare
d13be43
to
a242a05
Compare
0d58750
to
ffd2df3
Compare
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.
LGTM ✅ Nice PR :)
Can you please squash your commits since they do not seem to be atomic ?
ffd2df3
to
1cd672f
Compare
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.
Looks good to me 👍
6a88576
to
b1b7f85
Compare
b1b7f85
to
b1862de
Compare
Closes #7912