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

Provide more descriptive exception messages. #89

Merged
merged 1 commit into from
Apr 5, 2012

Conversation

djanowski
Copy link
Contributor

The current solution doesn't provide any useful information for notifications, etc.

@goofrider goofrider mentioned this pull request Feb 24, 2012
@dvdplm
Copy link

dvdplm commented Mar 28, 2012

+1

@djanowski
Copy link
Contributor Author

@sferik any thoughts?

@sferik
Copy link
Contributor

sferik commented Apr 5, 2012

Seems good. Merging.

sferik added a commit that referenced this pull request Apr 5, 2012
Provide more descriptive exception messages.
@sferik sferik merged commit 1838723 into oauth-xx:master Apr 5, 2012
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.

3 participants