Merge pull request 'fix retVal squash logic so remote vals dont override local vals and get sent up' (#84) from squshRet into trunk
continuous-integration/drone/push Build is pending Details

Reviewed-on: #84
Reviewed-by: Sarah Jamie Lewis <sarah@openprivacy.ca>
This commit is contained in:
Dan Ballard 2022-04-18 19:57:15 +00:00
commit c32d5082e4
1 changed files with 1 additions and 2 deletions

View File

@ -527,8 +527,7 @@ func (eh *EventHandler) handleProfileEvent(ev *EventProfileEnvelope) string {
return ""
}
}
if val, err := profile.GetConversationAttribute(conversation.ID, attr.LocalScope.ConstructScopedZonedPath(zone.ConstructZonedPath(path))); err != nil && val != "" {
if val, err := profile.GetConversationAttribute(conversation.ID, attr.LocalScope.ConstructScopedZonedPath(zone.ConstructZonedPath(path))); err == nil || val != "" {
// we have a locally set override, don't pass this remote set public scope update to UI
return ""
}