Skip to content

Commit d92139f

Browse files
authored
Merge branch 'master' into multi-embed-fix
2 parents a64989f + e976f60 commit d92139f

File tree

88 files changed

+10770
-4132
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

88 files changed

+10770
-4132
lines changed

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
node_modules/
2-
bower_components/
32
coverage/
43
docs
54
typings/

CONTRIBUTING.md

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@ The build and tests use webpack to compile all the source modules into one bundl
5858
## Running the demo
5959
Navigate to `/demo` directory
6060

61-
Install bower dependencies:
61+
Install npm dependencies:
6262
```
63-
bower install
63+
npm install
6464
```
6565

6666
Serve the demo directory:
@@ -84,4 +84,12 @@ npm run gulp -- build:docs
8484
If the docs are correct then you may publish them to gh-pages using this command
8585
```
8686
npm run gulp -- ghpages
87-
```
87+
```
88+
89+
## Known issues
90+
Running demo fails with an error ERR_INVALID_REDIRECT
91+
This happens due to version 10 of http-server. To solve the problem, please install [email protected] globally using:
92+
93+
```
94+
npm install -g [email protected]
95+
```

README.md

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,20 @@ JavaScript library for embedding Power BI into your apps.
33

44
[![Build Status](https://img.shields.io/travis/Microsoft/PowerBI-JavaScript/master.svg)](https://travis-ci.org/Microsoft/PowerBI-JavaScript)
55
[![NPM Version](https://img.shields.io/npm/v/powerbi-client.svg)](https://www.npmjs.com/package/powerbi-client)
6-
[![Bower Version](https://img.shields.io/bower/v/powerbi-client.svg)](https://bower.io/search/?q=powerbi-client)
76
[![Nuget Version](https://img.shields.io/nuget/v/Microsoft.PowerBI.JavaScript.svg)](https://www.nuget.org/packages/Microsoft.PowerBI.JavaScript/)
87
[![NPM Total Downloads](https://img.shields.io/npm/dt/powerbi-client.svg)](https://www.npmjs.com/package/powerbi-client)
98
[![NPM Monthly Downloads](https://img.shields.io/npm/dm/powerbi-client.svg)](https://www.npmjs.com/package/powerbi-client)
109
[![GitHub tag](https://img.shields.io/github/tag/microsoft/powerbi-javascript.svg)](https://github.com/Microsoft/PowerBI-JavaScript/tags)
1110
[![Gitter](https://img.shields.io/gitter/room/Microsoft/PowerBI-JavaScript.svg)](https://gitter.im/Microsoft/PowerBI-JavaScript)
1211

1312
## Wiki
14-
See the [wiki](https://github.com/Microsoft/PowerBI-JavaScript/wiki) for more details about embedding, service configuration, setting default page, page navigation, dynamically applying filters, and more.
13+
See the [wiki](https://github.com/Microsoft/PowerBI-JavaScript/wiki) for more details about embedding, service configuration, setting a default page, page navigation, dynamically applying filters, and more.
1514

1615
## Code Docs
1716
See the [code docs](https://microsoft.github.io/PowerBI-JavaScript) for detailed information about classes, interfaces, types, etc.
1817

1918
## Demo
20-
New [live demo](https://microsoft.github.io/PowerBI-JavaScript/demo) for sample application using the powerbi-client library in scenarios such as page navigation, applying filters, updating settings, and more.
19+
New [live demo](https://microsoft.github.io/PowerBI-JavaScript/demo) for a sample application using the `powerbi-client` library in scenarios such as page navigation, applying filters, updating settings, and more.
2120

2221
## Installation
2322

@@ -29,29 +28,25 @@ Install from NPM:
2928

3029
`npm install --save powerbi-client`
3130

32-
Install from Bower:
33-
34-
`bower install powerbi-client --save`
35-
3631
Installing beta versions:
3732

3833
`npm install --save powerbi-client@beta`
3934

4035
## Include the library via import or manually
4136

42-
Ideally you would use module loader or compilation step to import using ES6 modules as:
37+
Ideally you would use a module loader or a compilation step to import using ES6 modules as:
4338

4439
```javascript
4540
import * as pbi from 'powerbi-client';
4641
```
4742

48-
However, the library is exported as a Universal Module and the powerbi.js script can be included before your apps closing `</body>` tag as:
43+
However, the library is exported as a Universal Module and the powerbi.js script can be included before your app's closing `</body>` tag as:
4944

5045
```html
51-
<script src="/bower_components/powerbi-client/dist/powerbi.js"></script>
46+
<script src="/powerbi-client/dist/powerbi.js"></script>
5247
```
5348

54-
When included directly the library is exposd as a global named 'powerbi-client'.
55-
There is also another global `powerbi` which is an instance of the service.
49+
When included directly, the library is exposed as a global named `powerbi-client`.
50+
There is also another global named `powerbi` which is an instance of the service.
5651

5752

bower.json

Lines changed: 0 additions & 30 deletions
This file was deleted.

demo/bower.json

Lines changed: 0 additions & 26 deletions
This file was deleted.

demo/code-demo/scripts/session_utils.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@ const SessionKeys = {
66
EmbedId : "embedId",
77
GroupId : "groupId",
88
IsSampleReport: "isSampleReport",
9-
QnaQuestion: "qnaQuestion"
9+
QnaQuestion: "qnaQuestion",
10+
EntityIsAlreadyEmbedded: "EntityIsAlreadyEmbedded",
1011
};
1112

1213
function GetParameterByName(name, url) {
@@ -45,7 +46,7 @@ function SetTextBoxesFromSessionOrUrlParam(accessTokenSelector, embedUrlSelector
4546
{
4647
accessToken = GetSession(SessionKeys.AccessToken);
4748
}
48-
49+
4950
var embedUrl = GetParameterByName(SessionKeys.EmbedUrl);
5051
if (!embedUrl)
5152
{

demo/package.json

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,18 @@
2525
"ignore": [
2626
"**/.*",
2727
"node_modules",
28-
"bower_components",
2928
"test",
3029
"tests"
3130
],
3231
"dependencies": {
33-
"bootstrap": "^3.3.6",
32+
"bootstrap": "^4.1.2",
33+
"ecstatic": "^3.3.1",
3434
"es6-promise": "^3.2.2",
3535
"fetch": "^1.0.0",
36-
"http-server": "^0.10.0",
36+
"http-server": "^0.9.0",
3737
"jquery": "^3.1.0",
38-
"powerbi-client": "^2.6.5",
38+
"powerbi-client": "file:..",
39+
"powerbi-report-authoring": "^1.0.0",
3940
"syntaxhighlighter": "4.0.1"
4041
},
4142
"devDependencies": {}

demo/v2-demo/docs.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
<div class="docs-line">
44
<div class="docs-links">
55
<h2>Getting started</h2>
6+
<a href="https://app.powerbi.com/embedsetup">Set up your Power BI embedding environment</a><br>
67
<a href="https://github.com/Microsoft/powerbi-javascript/wiki">Power BI JavaScript API wiki</a><br>
78
<a href="https://docs.microsoft.com/en-us/power-bi/developer/embedding">Power BI embedding documentation</a><br>
89
</div>
Lines changed: 3 additions & 0 deletions
Loading
Lines changed: 3 additions & 0 deletions
Loading

0 commit comments

Comments
 (0)