-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
Add Lucky Orange integration #175
Conversation
Codecov Report
@@ Coverage Diff @@
## master #175 +/- ##
==========================================
+ Coverage 94.14% 94.25% +0.11%
==========================================
Files 28 29 +1
Lines 1246 1270 +24
==========================================
+ Hits 1173 1197 +24
Misses 73 73
Continue to review full report at Codecov.
|
fb5ee72
to
9861063
Compare
I proof read the docs and they seem good to me. |
9861063
to
05213ae
Compare
Okay, I rebased the branch. So, we only need to decide whether we want to require explicit confirmation from Lucky Orange support on the included CDN URL, before merging. BTW, to implement this integration I simply copy+pasted & search+replaced the excellent Hotjar integration code, including the docs. And for the rebasing I made sure with a |
Adds a new integration for a service similar to Hotjar.
Fixes #172