CL Fisher
CL Fisher
@dorner. If I read the tests right, there are failures that are related to this change.
TODO onthis -- write up the very edge case (see the PR) as a more specific issue, and then close this one.
On the other hand, I'm a little leery about having such a big PR in a release along with a host of other things. *g*. Let's see if we can...
Proto issue added -- at a "this is happening in production" level, because it has in the last month. Closing this issue.
This *should* only affect admin things, though, right?
(nods) That makes me more at ease about it going in with a bunch of other stuff.
Hey @dorner, In the context that this PR 'removes the need for organization name in admin controllers", seeing this path when I logged as the superadmin surprised me. The organization...
Of course, it appears to be working *now*... which puts this back on my list to "kick" -- hopefully I'll find out how I got there.
I haven't found it for the dashboard, but this one, which is where you get to after you perform an action on a user in the organization view... seems like...
I'm happy with it from a functional pov now. Would like @awwaiid to take a look from a technical pov.