Merge pull request #324 from marclucraft/patch-1

Updated async attribute.

Author: jkasten2

onesignal-public.php

@@ -2,16 +2,6 @@
 
 defined('ABSPATH') or die('This page may not be accessed directly.');
 
-function add_async_for_script($url)
-{
-	if (strpos($url, '#asyncload') === false)
-	    return $url;
-	else if (is_admin())
-	    return str_replace('#asyncload', '', $url);
-	else
-	    return str_replace('#asyncload', '', $url)."' async='async";
-}
-
 class OneSignal_Public
 {
     public function __construct()
@@ -77,13 +67,11 @@ public static function onesignal_header()
             OneSignal_Public::insert_onesignal_stamp();
         } ?>
     <?php
-    add_filter('clean_url', 'add_async_for_script', 11, 1);
-
     if (defined('ONESIGNAL_DEBUG') && defined('ONESIGNAL_LOCAL')) {
-        wp_register_script('local_sdk', 'https://localhost:3001/sdks/OneSignalSDK.js#asyncload', array(), false, true);
+	wp_register_script('local_sdk', 'https://localhost:3001/sdks/OneSignalSDK.js', array(), '1.0.0', array('strategy' => 'async'));
         wp_enqueue_script('local_sdk');
     } else {
-        wp_register_script('remote_sdk', 'https://cdn.onesignal.com/sdks/OneSignalSDK.js#asyncload', array(), false, true);
+        wp_register_script('remote_sdk', 'https://cdn.onesignal.com/sdks/OneSignalSDK.js', array(), '1.0.0', array('strategy' => 'async'));
         wp_enqueue_script('remote_sdk');
     } ?>
     <script>