fix: added boolean return for global set to fix Proxy set trapped err…

…or (#134)

* fix: added boolean return for global set to fix Proxy set trapped error

* fox: added receiver paramer for setter

* remvoed Reflect as it caused issues with window proxy object

* Release 2.8.2

lib/global/index.js

@@ -19,20 +19,20 @@ var getGlobal = function () {
     global import in all bundle use cases
   */
   if (globalObj && globalObj.__esModule) {
-    const override = {
-      get: function (target, prop, receiver) {
+    const proxyGlobal = new Proxy(globalObj, {
+      get: function (target, prop) {
         if (prop === 'default') {
           return target
         }
 
-        return Reflect.get(receiver, prop, target)
+        return target[prop]
       },
 
       set: function (target, prop, value) {
-        Reflect.set(target, prop, value)
+        target[prop] = value
+        return true
       },
-    };
-    const proxyGlobal = new Proxy(globalObj, override)
+    })
     return proxyGlobal
   }
   return globalObj

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bv-ui-core",
-  "version": "2.8.1",
+  "version": "2.8.2",
   "license": "Apache 2.0",
   "description": "Bazaarvoice UI-related JavaScript",
   "repository": {