-
-
Notifications
You must be signed in to change notification settings - Fork 73
feat(nuxt): Set sendDefaultPii: true
by default
#1060
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
base: master
Are you sure you want to change the base?
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1060 +/- ##
=======================================
Coverage 32.12% 32.12%
=======================================
Files 133 133
Lines 15696 15696
Branches 1100 1100
=======================================
Hits 5042 5042
Misses 10637 10637
Partials 17 17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
d034ed4
to
1444136
Compare
"vue": "latest", | ||
"vue-router": "latest" | ||
}, | ||
"resolutions": { |
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.
do we need this for the test app to run? just curious, no action required
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.
Yes, it was prompting and breaking the E2E test flow before adding this resolution
1444136
to
75cb206
Compare
@Lms24, updated 👍 |
Resolves: #961