twenty icon indicating copy to clipboard operation
twenty copied to clipboard

From Sentry: GraphQL errors on deleteFromcompany: [{"message":"delete impacts too many records"}]

Open sentry[bot] opened this issue 1 year ago • 0 comments
trafficstars

What

pg_graphql seems to have an "atMost" parameter. We should probably increase this parameter (100?) and make sure this is catched and returned with a friendly user message at the application layer - and not an internal server error.

Sentry

Sentry Issue: PROD-2F

InternalServerErrorException: GraphQL errors on deleteFromcompany: [{"message":"delete impacts too many records"}]
  File "/app/packages/twenty-server/dist/src/workspace/workspace-query-runner/workspace-query-runner.service.js", line 143, in WorkspaceQueryRunnerService.parseResult
    throw new common_1.InternalServerErrorException(`GraphQL errors on ${command}${targetTableName}: ${JSON.stringify(errors)}`);
  File "/app/packages/twenty-server/dist/src/workspace/workspace-query-runner/workspace-query-runner.service.js", line 118, in WorkspaceQueryRunnerService.deleteMany
    return (_a = this.parseResult(result, targetTableName, 'deleteFrom')) === null || _a === void 0 ? void 0 : _a.records;
...
(1 additional frame(s) were not displayed)

sentry[bot] avatar Dec 23 '23 08:12 sentry[bot]