Don't overwrite UserVisibleError when updating selectors fails (#4334)

We would return a UserVisibleError when updating selectors, but then
overwrite it in the caller who caled the validation, which resulted in
the server returning a 500 HTTP code to the client.

internal/profiles/service.go

@@ -285,7 +285,7 @@ func (p *profileService) UpdateProfile(
 	}
 
 	if err := p.updateSelectors(ctx, updatedProfile.ID, qtx, profile.GetSelection()); err != nil {
-		return nil, status.Errorf(codes.Internal, "error updating profile: %v", err)
+		return nil, err
 	}
 
 	logger.BusinessRecord(ctx).Profile = logger.Profile{Name: updatedProfile.Name, ID: updatedProfile.ID}