-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
feat(common): add error options object #10460
Merged
kamilmysliwiec
merged 38 commits into
nestjs:master
from
thiagomini:feature/10392-http-exceptions-cause-with-custom-message
Nov 7, 2022
Merged
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
b5fe06d
feat(common): add error options object
thiagomini f5edb0f
chore(common): add deprecated warning
thiagomini 587fbae
feat(common): add error cause option
thiagomini 071c4c5
refactor(common): change parameter assignment
thiagomini d63b6df
refactor(common): rearrange test blocks
thiagomini 7e763fa
refactor(common): rename method parameter
thiagomini 1a8ada5
test(common): add http exception test
thiagomini 4ded779
refactor(common): add description attribute
thiagomini c5c818e
chore(common): change deprecated message
thiagomini 60cb953
feat(common): add error cause option
thiagomini 7a71035
refactor(common): extract description and options
thiagomini a4a8557
refactor(common): extract description and options
thiagomini 5a186af
docs(common): add static method docs
thiagomini 86c5084
test(common): test error cause option
thiagomini 472c711
test(common): aggregate exception tests
thiagomini 88ac6aa
feat(common): add error cause option
thiagomini 1dff9c0
docs(common): update exception docs
thiagomini c3ced1d
feat(common): add error cause option
thiagomini f629dc1
feat(common): add error cause option
thiagomini bd7daf1
feat(common): add error cause option
thiagomini 0328a5c
feat(common): add error cause option
thiagomini 11d2763
feat(common): add error cause option
thiagomini b815143
feat(common): add error cause option
thiagomini 789e99c
feat(common): add error cause option
thiagomini cd44ff3
feat(common): add error cause option
thiagomini 3a10258
feat(common): add error cause option
thiagomini d569aaa
feat(common): add error cause option
thiagomini 2416df6
feat(common): add error cause option
thiagomini dfc28fc
feat(common): add error cause option
thiagomini d40ebd1
feat(common): add error cause option
thiagomini 5031f41
feat(common): add error cause option
thiagomini e8abd9a
feat(common): add error cause option
thiagomini f62acac
feat(common): add error cause option
thiagomini b4aca5d
feat(common): add error cause option
thiagomini 379b82c
feat(common): add error cause option
thiagomini 971ce5f
test(common): reuse error cause variable
thiagomini 45b6cc1
test(common): add http exception tests
thiagomini ab881e3
test(common): add http exception tests
thiagomini File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I renamed this parameter to reflect its usage better. When this is a string, it's used as the
message
attribute below, so this new name seemed a better fit.