cqrs icon indicating copy to clipboard operation
cqrs copied to clipboard

feat(): Improve query bus to throw error with query name

Open HerrSpeck opened this issue 1 year ago • 2 comments

I tried to stick to the changes made in the PR for the Command Handler Exceptions (#1492) .

PR Checklist

Please check if your PR fulfills the following requirements:

  • [ ] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
  • [ ] Tests for the changes have been added (for bug fixes / features)
  • [ ] Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • [ ] Bugfix
  • [X] Feature
  • [ ] Code style update (formatting, local variables)
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes
  • [ ] CI related changes
  • [ ] Other... Please describe:

What is the current behavior?

Issue Number: #1491

The Query Handler exceptions have the same problem as the Command Handler exceptions.

What is the new behavior?

Errors are thrown with the name of the query and not it's uuid.

Does this PR introduce a breaking change?

  • [ ] Yes
  • [X] No

Other information

HerrSpeck avatar Jul 13 '24 08:07 HerrSpeck