Merge pull request microsoft#10 from Microsoft/dev

Author: mattmazzola

.gitignore

@@ -2,3 +2,4 @@ node_modules/
 bower_components/
 coverage/
 typings/
+test/*.spec.js

CHANGELOG.md

@@ -0,0 +1,60 @@
+# 2.0.0-beta.1 (GA candidate)
+
+## Breaking
+
+- DOMContentLoaded handler is now opt-in instead of the default behavior
+  - Reasons:
+    - The primary use case will be using the core library within another library which may not have the DOM ready even if DOMContentLoaded has fired.
+    - Most developers using SPA applications will fetch embed data asynchronously and not know report data by the time the DOMContentLoaded has fired.
+  - How to opt-in to DOMContentLoaded:
+    - Call `enableAutoEmbed()` on an instance of the PowerBi service to add the event listener.
+      
+      Example:
+      ```
+      <script src="powerbi.js"></script>
+      <script>
+      powerbi.enableAutoEmbed();
+      </script>
+      ```
+      
+      Alternately if you are creating an instance of the service you can pass a configuration parameter `autoEmbedOnContentLoaded`
+      
+      Example:
+      ```
+      var powerbiService = new Powerbi({ autoEmbedOnContentLoaded: true });
+      ```
+- `powerbi.get(element: HTMLElement)` is removed. Use `powerbi.embed(element: HTMLElement, config: IEmbedOptions = {})`.
+  - Reasons:
+    - powerbi.embed performed the same function and is more semantic.
+- Embed urls must be provided by the server and will not be constructed by the components. This implies that the attributes `powerbi-report` will no longer be used.
+  - Reasons:
+  
+      The construction of these urls was unreliable since it dependeded on assumptions about server configuration (target environment, component type, etc).
+      Since url would be incorrect in some cases it could cause trouble for developers. Also, since the sever is already returning access tokens it's trival for the server to also provide embed urls and this reduces complexity.
+  
+      Previously you could supply the embed information in two ways:
+      
+      1. Using report id:
+      
+      `<div powerbi-embed powerbi-report="5dac7a4a-4452-46b3-99f6-a25915e0fe55" powerbi-access-token="..."></div>`
+      
+      This would implicitly construct the embed url to be something like: `https://embedded.powerbi.com/reports/5dac7a4a-4452-46b3-99f6-a25915e0fe55`
+      However 
+      
+      2. Using embed url:
+      
+      `<div powerbi-embed-url="https://embedded.powerbi.com/reports/5dac7a4a-4452-46b3-99f6-a25915e0fe55" powerbi-access-token="..."></div>`
+      
+      Now only the latter options (#2) is supported.
+      
+- Embed url attribute changed from `powerbi-embed` to `powerbi-embed-url`
+- Component type is specified by attribute `powerbi-type`. Use `powerbi-type="report"` instead of applying the attribute `powerbi-report`
+- Configuration option attributes all start with prefix `powerbi-options-`. Example (`powerbi-options-filter`)
+
+## Changes
+
+- Fix bug to prevent memory leak of holding references to iframe elements which had been removed from the DOM.
+- Detect overwriting container with new component and either throw error or properly cleanup and replace old component based on `config.overwrite` setting. 
+- Fix bug with prematurely attempting to parse post message data until it is known that it originated from embedded iframe.
+  
+# 1.0.0 (Preview)

dist/powerbi.d.ts

@@ -7,6 +7,12 @@ export interface IPowerBiConfiguration {
     onError?: (error: any) => any;
 }
 
+declare global {
+    interface Window {
+        Powerbi: typeof PowerBi;
+    }
+}
+
 export class PowerBi {
     /**
      * List of components this service can embed.
@@ -42,6 +48,12 @@ export class PowerBi {
      * If component has already been created and attached to eleemnt simply return it to prevent creating duplicate components for same element.
      */
     embed(element: IPowerBiElement, config?: IEmbedOptions): Embed;
+    /**
+     * Adds event handler for DOMContentLoaded which finds all elements in DOM with attribute powerbi-embed-url
+     * then attempts to initiate the embed process based on data from other powerbi-* attributes.
+     * (This is usually only useful for applications rendered on by the server since all the data needed will be available by the time the handler is called.)
+     */
+    enableAutoEmbed(): void;
     /**
      * Remove component from the list of embedded components.
      */
@@ -53,7 +65,7 @@ export class PowerBi {
      *
      * If an error occurs when parsing event.data call error handler provided during configuration.
      */
-    onReceiveMessage(event: MessageEvent): void;
+    private onReceiveMessage(event: MessageEvent): void;
 }
 
 
@@ -65,11 +77,15 @@ export interface IEmbedOptions {
     embedUrl?: string;
     webUrl?: string;
     name?: string;
+    filter?: any;
     filterPaneEnabled?: boolean;
     getGlobalAccessToken?: () => string;
     overwrite?: boolean;
 }
 declare abstract class Embed {
+    public static embedUrlAttribute: string;
+    public static accessTokenAttribute: string;
+    public static typeAttribute: string;
     /**
      * Default options for embeddable component.
      */