-
Notifications
You must be signed in to change notification settings - Fork 173
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
IETF API-Problem specification for error messages #49
Conversation
@cpliakas - how cool is that? :) |
Tests fixed. |
public function getType() { | ||
// Depending on the error code we'll return a different URL. | ||
$url = 'http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html'; | ||
$sections = array( |
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.
You are crazy! ;)
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.
Some HTML c&p and text editor magic did the trick, and it adheres to the proposal! Also, bonus for linking to W3.org
.
Code looks good, I think before merge it's worth adding a line or two in the README - care to do it? |
Right! The section about Error Hnadling needs to be updated. On Wed, May 28, 2014 at 8:13 PM, Amitai Burstein
Mateu Aguiló Bosch. w. lb.cm/e0ipso |
Added README notes. |
This should be good to merge. |
Merged, thanks! |
@mateu-aguilo-bosch Great work! @amitaibu, thanks for being open to this change and committing! |
Implements #27.