zengenuity / civicrm_groupprice

CiviCRM extension that allows you to limit the display of individual prices in price sets to specific groups. This can be used to implement members-only pricing for events, as well as other use cases.
Other
3 stars 9 forks source link

Static Groups not allowed #14

Open rbaugh opened 2 years ago

rbaugh commented 2 years ago

The Readme states that static and smart groups can be used, but the code is only checking for smart groups. If the group doesn't have the saved_search_id value, which is only set on Smart Groups, then it will exclude static groups.

Also, this seems to only want to validate if the person is logged in. This can be a challenge if you want to send out an email with a checksum in the link to a contact for an event and because they already attended an event, you could provide them a discount. Looking for the checksum could extend this out as well.

rbaugh commented 2 years ago

I think the issue of static groups not working might be related to issue #13.