walk the tree of pages to find an index

[mansona]

This is needed because we currently have the ability to have arbitrary nesting of routes. I am personally not a fan of making this so complex but with the current feature set it's necessary 😞

[jenweber]

I think using recursion makes sense here, and your code comments make it easy to follow.

Failing tests are for beta and canary of Ember, for a method you aren't using directly anywhere in this repo. I wonder if perhaps this should be merged after setting beta to be allowed to fail, and then we fix that separately.

[mansona]

@jenweber so sorry, I forgot to mention that beta and canary were failing because of emberjs/ember.js#18693 which is now fixed. I re-pushed my change to kick off CI again and it should be ready to merge 👍

[mansona]

@jenweber it looks like I spoke too soon 🤔 It looks like CI isn't failing because of the known bug I was linking to, but instead it's failing because of a different regression in Ember 🙈 I will try to investigate a little bit if I can

[mansona]

@jenweber I ended up fixing CI by updating ember because there were a few strange things going on with older ember-try scenarios.

If you're still ok with this can you review and merge? thanks 🎉

[jenweber]

Looks good! Thank you Chris!