Apollo Devtools extension download message check if does not exist

[alexandermirzoyan]

This PR is about having a working check before logging "Download the Apollo DevTools for a better development..." message.

[apollo-cla]

@alexandermirzoyan: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

[netlify]

👷 Deploy request for apollo-client-docs pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	45489b9

[changeset-bot]

⚠️ No Changeset found

Latest commit: 45489b9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[phryneas]

Is this in itself still a problem for you with the latest version of Apollo Client? The check itself has recently been changed to not trigger false positives.

[alexandermirzoyan]

I thought !(window as any).__APOLLO_DEVTOOLS_GLOBAL_HOOK__ this line was not working. Seems it's added after v3.7.0, as we use that version in our codebase.

[phryneas]

The important part was added in 3.8.2 (I think) - a setTimeout before making the check, to give the devtools time to initialize before checking if they are there.
Please give the latest version a test - this might not be necessary anymore.

[alexandermirzoyan]

Great! Thank you.
So I am going to close this PR

[phryneas]

Thanks a ton for the PR though, I really appreciate the time you put into this :)