Skip to content

Commit

Permalink
fix(aot): Remove AuthBackend param for AOT support
Browse files Browse the repository at this point in the history
  • Loading branch information
davideast committed Dec 26, 2016
1 parent 4630a86 commit f875360
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "angularfire2",
"version": "2.0.0-beta.6",
"version": "2.0.0-beta.7-pre",
"description": "",
"main": "bundles/angularfire2.umd.js",
"module": "index.js",
Expand Down Expand Up @@ -28,7 +28,7 @@
"type": "git",
"url": "git+https://github.com/angular/angularfire2.git"
},
"author": "jeffbcross",
"author": "jeffbcross,davideast",
"license": "MIT",
"bugs": {
"url": "https://github.com/angular/angularfire2/issues"
Expand Down
2 changes: 1 addition & 1 deletion src/angularfire2.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export function _getWindowLocation(){
}

export function _getAuthBackend(app: firebase.app.App): FirebaseSdkAuthBackend {
return new FirebaseSdkAuthBackend(app, false);
return new FirebaseSdkAuthBackend(app);
}

export function _getDefaultFirebase(config){
Expand Down
3 changes: 1 addition & 2 deletions src/auth/firebase_sdk_auth_backend.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@ export class FirebaseSdkAuthBackend extends AuthBackend {
* https://github.com/angular/angular/issues/12631
* https://github.com/angular/angularfire2/issues/653
**/
constructor( @Inject(FirebaseApp) _fbApp: any,
private _webWorkerMode = false) {
constructor(@Inject(FirebaseApp) _fbApp: any) {
super();
this._fbAuth = _fbApp.auth();
}
Expand Down

1 comment on commit f875360

@kleeb
Copy link

@kleeb kleeb commented on f875360 Mar 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is that?
after this change I am getting

require is not defined

Please sign in to comment.