At some point, I introduced a fast-path for subscription upsert where if no conditions are provided, we fail to check for constraint errors and wrap them with ErrInvalidArgument.
This change makes it so that we correctly check for constraint errors in both paths.
At some point, I introduced a fast-path for subscription upsert where if no conditions are provided, we fail to check for constraint errors and wrap them with
ErrInvalidArgument.This change makes it so that we correctly check for constraint errors in both paths.
Fixes https://sourcegraph.sentry.io/issues/5717030715/?referrer=slack&alert_rule_id=15230143&alert_type=issue
Test plan
DB tests