-
-
Notifications
You must be signed in to change notification settings - Fork 17
Add solution providers #127
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
base: master
Are you sure you want to change the base?
Changes from 1 commit
f09026b
85ec4f0
884f7be
a2574a3
b67993d
a2b5afd
5c2c4b0
7c81988
6e2ceb2
b5e5ddf
dd29662
8bf9840
ce78cfc
5d41ab0
e904abf
6371efe
4d1259a
8cfee00
850326e
a45baf8
ff4a1a0
3a8bcc7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -4,6 +4,10 @@ | |||||
|
|
||||||
| namespace Yiisoft\ErrorHandler\Solution; | ||||||
|
|
||||||
| /** | ||||||
| * The interface declares an adapter to render a solution for an event. | ||||||
| * Basically, it renders the error message as-is, but possible could render a button with click-to-fix action that will be handled by an HTTP request (with middleware) back to server. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| */ | ||||||
| interface SolutionProviderInterface | ||||||
|
xepozz marked this conversation as resolved.
vjik marked this conversation as resolved.
xepozz marked this conversation as resolved.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's not about any solution in the air. It's about the specific contract between web, user, application and error.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Interface should define format of solution. Users of interface should know about format of solution.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. HTML for sure |
||||||
| { | ||||||
| public function supports(\Throwable $e): bool; | ||||||
|
xepozz marked this conversation as resolved.
|
||||||
|
|
||||||
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.