messaging options not being set about blackbaud-to-google-group-sync HOT 2 OPEN

Appears that listMembers() returns an array of Member-like objects, but not actually members. They are missing the delivery_settings field. Which means that to get the delivery settings for each member, each member would need to be individually queried per group, which is prohibitively resource expensive.

https://api.github.com/groton-school/blackbaud-to-google-group-sync/blob/6c30da3acbb2f3f1756fa7df850b0dc4daa18750/src/server/Workflows/sync.php#L125

            /*
            if ($gMember->getDeliverySettings() != $deliverySettings) {
                $oldDeliverySettings = $gMember->getDeliverySettings();
                $gMember->setDeliverySettings($deliverySettings);
                $gMember = $directory->members->update(
                    $bbGroup->getParamEmail(),
                    $gMember->getId(),
                    $gMember
                );
                $listProgress->setStatus(
                    "Updated {$gMember->email} delivery_settings from to '{$gMember->getDeliverySettings()}'"
                );
            }
            */

from blackbaud-to-google-group-sync.

At the moment, no longer checking existing member's delivery settings, but setting new members delivery settings. This holds until the delivery settings on a group are changed in Blackbaud, necessitating the update of the members' delivery settings in Google, and -- likewise -- does not check for changes in Google that don't match the existing Blackbaud setting.

For our current use cases, this is mostly safe, but annoying.

from blackbaud-to-google-group-sync.