Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Fix empty notification group payload #548

Merged
merged 2 commits into from
May 11, 2022

Conversation

djabarovgeorge
Copy link
Contributor

Closes #540

@vercel
Copy link

vercel bot commented May 9, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
docs ✅ Ready (Inspect) Visit Preview May 11, 2022 at 6:22AM (UTC)

@linear
Copy link

linear bot commented May 9, 2022

NV-381 Error: Bad Request Exception

Sentry Issue: API-P

Error: Bad Request Exception
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_4ab6445a855071205dd8b6fcfe0156b2/node_modules/@nestjs/common/pipes/validation.pipe.js", line 89, col 20, in ValidationPipe.exceptionFactory
    return new http_error_by_code_util_1.HttpErrorByCode[this.errorHttpStatusCode](errors);
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_4ab6445a855071205dd8b6fcfe0156b2/node_modules/@nestjs/common/pipes/validation.pipe.js", line 65, col 30, in ValidationPipe.transform
    throw await this.exceptionFactory(errors);
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_4ab6445a855071205dd8b6fcfe0156b2/node_modules/@nestjs/common/pipes/validation.pipe.js", in runMicrotasks
  File "node:internal/process/task_queues", line 96, col 5, in processTicksAndRejections
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_cf22c14faf7a8288645a80ae61717d6f/node_modules/@nestjs/core/router/router-execution-context.js", line 147, col 23, in resolveParamValue
    ? await this.getParamValue(value, { metatype, type, data }, pipes.concat(paramPipes))
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_4ab6445a855071205dd8b6fcfe0156b2/node_modules/@nestjs/common/pipes/validation.pipe.js", in Promise.all
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_cf22c14faf7a8288645a80ae61717d6f/node_modules/@nestjs/core/router/router-execution-context.js", line 150, col 13, in pipesFn
    await Promise.all(paramsOptions.map(resolveParamValue));
  File "/usr/src/app/node_modules/.pnpm/@[email protected]_cf22c14faf7a8288645a80ae61717d6f/node_modules/@nestjs/core/router/router-execution-context.js", line 37, col 30, in <anonymous>
    fnApplyPipes && (await fnApplyPipes(args, req, res, next));

@gitpod-io
Copy link

gitpod-io bot commented May 9, 2022

@djabarovgeorge djabarovgeorge merged commit 9a306f8 into main May 11, 2022
@djabarovgeorge djabarovgeorge deleted the NV-381-Error-Bad-Request-Exception branch May 11, 2022 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Empty notification group on creation of notification-templates
2 participants