Merge pull request #133 from Flagsmith/chores/replace_inline_event_trigger

Use un-minified event-trigger, tidy TypeScript

Author: kyle-ssg

.gitignore

@@ -39,3 +39,4 @@ buck-out/
 \.buckd/
 android/app/libs
 android/keystores/debug.keystore
+*.js.map

examples/nextjs/package.json

@@ -9,7 +9,7 @@
     "lint": "next lint"
   },
   "dependencies": {
-    "flagsmith-es": "3.4.0",
+    "flagsmith-es": "3.6.0",
     "lodash-es": "^4.17.21",
     "next": "12.1.0",
     "next-transpile-modules": "^9.0.0",

flagsmith-core.ts

@@ -133,7 +133,6 @@ const Flagsmith = class {
                 this.getJSON(api + 'identities/?identifier=' + encodeURIComponent(identity)),
             ])
                 .then((res) => {
-                    // @ts-ignore
                     this.withTraits = false
                     handleResponse(res[0])
                     if (resolve && !resolved) {
@@ -148,8 +147,7 @@ const Flagsmith = class {
                 this.getJSON(api + "flags/")
             ])
                 .then((res) => {
-                    // @ts-ignore
-                    handleResponse({ flags: res[0] })
+                    handleResponse({ flags: res[0], traits:null })
                     if (resolve && !resolved) {
                         resolved = true;
                         resolve();
@@ -281,11 +279,11 @@ const Flagsmith = class {
             }
 
             if (enableDynatrace) {
-                // @ts-ignore
+                // @ts-expect-error Dynatrace's dtrum is exposed to global scope
                 if (typeof dtrum === 'undefined') {
                     console.error("You have attempted to enable dynatrace but dtrum is undefined, please check you have the Dynatrace RUM JavaScript API installed.")
                 } else {
-                    // @ts-ignore
+                    // @ts-expect-error Dynatrace's dtrum is exposed to global scope
                     this.dtrum = dtrum;
                 }
             }

flagsmith-es/index.js

@@ -1,6 +1,6 @@
 {
   "name": "flagsmith-es",
-  "version": "3.5.0",
+  "version": "3.6.0",
   "description": "Feature flagging to support continuous development. This is an esm equivalent of the standard flagsmith npm module.",
   "main": "./index.js",
   "type": "module",