veliovgroup / flow-router

🚦 Carefully extended flow-router for Meteor
https://packosphere.com/ostrio/flow-router-extra
BSD 3-Clause "New" or "Revised" License
202 stars 29 forks source link

Nested groups triggersEnter are not fired #16

Closed kpala closed 7 years ago

kpala commented 7 years ago

It seems nested groups do not fire all triggersEnter functions. I'm using version 3.1.3.

Consider the following example:

const loginGroup = FlowRouter.group({
    triggersEnter: [() => {
        console.log('login group');
    }]
});

const rootGroup = loginGroup.group({
    triggersEnter: [() => {
        console.log('root group');
    }]
});

rootGroup.route('/', {
    action() {
        ...
    },
    triggersEnter: [() => {
        console.log('root route');
    }]
});

The output in console is then:

root group
root route

while with kadira:flow-router@2.12.1 it is (as it should be):

login group
root group
root route
dr-dimitru commented 7 years ago

Hello @kpala ,

Sorry for late response. Thank you a lot for submitting this issue.

This should be fixed in v.3.2.0, please upgrade and let me know if it work for you.