-
Notifications
You must be signed in to change notification settings - Fork 0
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
Staging #27
Conversation
nathang21
commented
Oct 5, 2017
•
edited
Loading
edited
- Enable feed of Shopify order data to Google Customer reviews to build our brand trust reputation.
CodeFactor found an issue: Duplicate Code It's currently on: This should be easily fixable in a future PR if we externalize our Returnly scripts (along with CSS) on a CDN. For documentation purposes in the meantime I think this duplicate code is reasonable. #29 |
// Smooch web messenger | ||
function loadScript(src, callback) { var s, r, t; r = false; s = document.createElement('script'); s.type = 'text/javascript'; s.src = src; s.onload = s.onreadystatechange = function() { if ( !r && (!this.readyState || this.readyState == 'complete') ) { r = true; callback(); } }; t = document.getElementsByTagName('script')[0]; t.parentNode.insertBefore(s, t); } loadScript('https://cdn.smooch.io/smooch.min.js', function() { | ||
|
||
// setting variable to chat button on sizing tab |
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.
#28 We need to fix this
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 except smooch extra lines of code