feat: Added onLoad callback to showReportDialog (getsentry#2061)

leroydev · HazAT · commit b22e3592d176 · 2019-05-10T11:19:46.000+02:00
* feat: Added onLoad callback to showReportDialog

* meta: Changelog
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- [browser] - feat: Added `onLoad` callback to `showReportDialog`
+
 ## 5.2.0
 
 - [opentracing] ref: Removed opentracing package
diff --git a/packages/browser/src/client.ts b/packages/browser/src/client.ts
@@ -28,6 +28,7 @@ export interface ReportDialogOptions {
   errorGeneric?: string;
   errorFormEntry?: string;
   successMessage?: string;
+  onLoad?(): void;
 }
 
 /**
@@ -99,6 +100,11 @@ export class BrowserClient extends BaseClient<BrowserBackend, BrowserOptions> {
     const script = document.createElement('script');
     script.async = true;
     script.src = new API(dsn).getReportDialogEndpoint(options);
+
+    if (options.onLoad) {
+      script.onload = options.onLoad;
+    }
+
     (document.head || document.body).appendChild(script);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ export interface ReportDialogOptions {`
`28`	`28`	`errorGeneric?: string;`
`29`	`29`	`errorFormEntry?: string;`
`30`	`30`	`successMessage?: string;`
	`31`	`+ onLoad?(): void;`
`31`	`32`	`}`
`32`	`33`
`33`	`34`	`/**`
`@@ -99,6 +100,11 @@ export class BrowserClient extends BaseClient<BrowserBackend, BrowserOptions> {`
`99`	`100`	`const script = document.createElement('script');`
`100`	`101`	`script.async = true;`
`101`	`102`	`script.src = new API(dsn).getReportDialogEndpoint(options);`
	`103`	`+`
	`104`	`+ if (options.onLoad) {`
	`105`	`+ script.onload = options.onLoad;`
	`106`	`+ }`
	`107`	`+`
`102`	`108`	`(document.head \|\| document.body).appendChild(script);`
`103`	`109`	`}`
`104`	`110`	`}`