Just did a simple smoke screen test of the dummy app with:
ActiveAdmin.register Post do
menu parent: "Users"
The item doesn't show up in the menu at all. The test suite passes with flying colors, so need to add a failing test as part of this as well. Definitely introduced as part of #1967 however.
I need to rebase my PR (#1488) anyway, so I'll likely be investigating this as well.
I've written some specs & a feature that fails finally, so just trying to get the fix in place & then I'll convert this to a PR.
Above pull request resolves this issue.
Tweaks suggested by @Daxter
Fixes #1986 - Menu should register child regardless of AA resource lo…
* ActiveAdmin::MenuItem#dom_id removed, wasn't needed
* Load order is no longer important in registering menu items