fix(sync-teams): 'IS NULL' instead of '= NULL'

mautrix / slack

A Matrix-Slack puppeting bridge

GNU Affero General Public License v3.0

61 stars 17 forks source link

fix(sync-teams): 'IS NULL' instead of '= NULL' #52

Closed dandehavilland closed 1 month ago

dandehavilland commented 1 month ago

Correctly removes user_team records that have null tokens.

The existing implementation of using =NULL in the query always results in a false negative. Correcting the syntax ensures the records without tokens are correctly deleted when calling SyncTeams.

tulir commented 1 month ago

This doesn't apply to the refactor and doesn't seem worth fixing for the old version