An example with Iron-Router about meteor-roles

[alanning]

Got a basic Iron-Router example going but it has some rough edges that I don't know how to fix.

https://github.com/alanning/meteor-roles/tree/iron-router

A lot of unknowns:

• how to set a layout in a before filter
• how to compose multiple filters
• why does notFoundTemplate in Router.configure not handle 404s
• why is Loading template continually displayed after login

Feel free to jump in and play around. Be sure to submit pull requests when you figure something out!

[alanning]

Well fixed the Loading template issue at least by using this.render('foo') rather than this.template = foo.

[alanning]

merged into master branch. still has rough edges but works.

[tony-kerz]

hey adrian,

thanks for the iron-router example, i'm trying to work thru that now.

with the disclosure that i'm not a JS expert, one thing that i wanted to ask about was this line:

https://github.com/alanning/meteor-roles/blob/master/examples/iron-router/client/routing.js#L85

the iron-router doc shows only a layoutTemplate option vs a layout option, should this be layoutTemplate here?

regards,
tony

[alanning]

Hi Tony,

Thanks for the email. Actually I think that is a change to the iron-router
api so this example probably needs to be updated. I wrote it a while ago
and iron-router has evolved quite a bit since then. (A good thing.)
That's actually why I have't publicized the example in the readme yet.

If you have time to get the example working with the latest iron-router, I
would love a pull request. :-)

Cheers,
Adrian

On Thu, Dec 19, 2013 at 11:19 AM, tony kerz [email protected]:

hey adrian,

thanks for the iron-router example, i'm trying to work thru that now.

with the disclosure that i'm not a JS expert, one thing that i wanted to
ask about was this line:

https://github.com/alanning/meteor-roles/blob/master/examples/iron-router/client/routing.js#L85

the iron-router doc shows only a layoutTemplate option vs a layoutoption, should this be
layoutTemplate here?

regards,
tony

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/16#issuecomment-30942000
.