UX: drop the `automatic_membership_retroactive` column from groups model. (PR #9430)

Instead, display the automatic membership user count details in a bootbox alert.

GitHub

I think something is wrong with your linter, as it seems to be adding many trailing commas to the javascript that seem to have nothing to do with this commit? Could you fix those please?

  data.automatic_membership_email_domains = email_domains;
    data.id = group_id;

I don’t like how domains is inserted here as it looks like an opportunity for SQL injection. Can you change it to be parameter instead?

(I realize the old code did this too, but this is an opportunity to harden our app.)

Looks like you left in debugging by accident :slight_smile:

Looks good now! Thanks.