remove unused parameter (no other calls to this fn)
fix membership_type to be editable within tests so we can address test failure