Comment 16 for bug 1749475

Thanks for the quick feedback, Mike. I get where you are coming from, and I see the utility, it just feels like something still isn't sitting quite right for me as it stands. I'm sorry to be a pain! Hopefully we can at least talk through it a little more and see if some improvements can be found.

Starting small, I am wondering about the utility of the flags on Event Def. Group Member (sortable, holdings, external). I understand what they mean, but what part of the process is supposed to consult these? Only 'holdings' is used right now. We also already have event_params and environment. Maybe something better could be done here by extending in that direction instead? I don't have a proposal, just leery of a pool of hard-coded options at this layer. What do you think?

I will also keep thinking and see if I can't get a clearer-headed about it.