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

Adds wix detection #134

Merged

Conversation

@housseindjirdeh
Copy link
Collaborator

commented Mar 6, 2019

@housseindjirdeh housseindjirdeh force-pushed the housseindjirdeh:adds-wix-detection branch from 7fa1c29 to 3527d9b Mar 7, 2019

@housseindjirdeh housseindjirdeh merged commit 2a252ff into johnmichel:master Mar 7, 2019

url: 'https://www.wix.com/',
npm: null,
test: function (win) {
if (win.wixBiSession) {

This comment has been minimized.

Copy link
@johnmichel

johnmichel Jun 29, 2019

Owner

Is there a secondary check that we could include here with &&?

This comment has been minimized.

Copy link
@paulirish

paulirish Jul 9, 2019

We got this check from the Wix engineers themselves, so I suspect this is sufficient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.