Change Details

It assumed the group default is 'all'. It should accept the 'default' field, and if that's omitted IMHO a safer default is '' (empty string, group has no effect). I'm doing part of this in https://gerrit.wikimedia.org/r/#/c/337546, but not the actual default part.

The backend already sends the default to the client. The client should accept the 'default' field, and if that's omitted IMHO a safer default is '' (empty string, group has no effect).

It assumed the group default is 'all'. It should accept the 'default' field, and if that's omitted IMHO a safer default is '' (empty string, group has no effect)The backend already sends the default to the client. I'm doing part of this in https://gerrit.wikimedia.org/r/#/c/337546The client should accept the 'default' field, and if that's omitted IMHO a safer default is '' (empty string, but not the actual default partgroup has no effect).