Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update to alpha 29 #24

Merged
merged 1 commit into from
Jul 7, 2015
Merged

Conversation

PatrickJS
Copy link
Contributor

updated to 29 and everything works

@@ -26,6 +24,6 @@ export class LoggedInRouterOutlet extends RouterOutlet {
if (!this.publicRoutes[url] && !localStorage.getItem('jwt')) {
instruction.component = Login;
}
super.activate(instruction);
return super.activate(instruction);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

you must return here otherwise you will have an error TypeError: Cannot read property 'then' of undefined

@PatrickJS
Copy link
Contributor Author

in alpha.30 we can switch to Http service (headers doesn't work in 29 atm without replacing XHRBackend)

@PatrickJS PatrickJS mentioned this pull request Jul 7, 2015
@@ -13,18 +13,17 @@ let template = require('./home.html');
selector: 'home'
})
@View({
template:`<style>${styles}</style>\n${template}`,
directives: [coreDirectives]
styles: [ styles ],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we have styles metadata now

Copy link
Contributor

Choose a reason for hiding this comment

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

yey!

@mgonto
Copy link
Contributor

mgonto commented Jul 7, 2015

Thanks :).

Merging!

mgonto added a commit that referenced this pull request Jul 7, 2015
@mgonto mgonto merged commit 1f3c37e into auth0-blog:master Jul 7, 2015
@PatrickJS PatrickJS deleted the alpha-29-update branch July 7, 2015 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants