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

Add proper messages to exceptions #17

Open
DVDPT opened this issue Apr 23, 2013 · 0 comments
Open

Add proper messages to exceptions #17

DVDPT opened this issue Apr 23, 2013 · 0 comments
Assignees

Comments

@DVDPT
Copy link

DVDPT commented Apr 23, 2013

I've been awhile struggling with an InvalidOperationException that turn out to be a browser navigation error, please add some kind of message when you throw some exception.

On the FacebookSessionClient class line 158:

            if (result.ResponseStatus == WebAuthenticationStatus.ErrorHttp)
            {
                throw new InvalidOperationException();
            }
            else if (result.ResponseStatus == WebAuthenticationStatus.UserCancel)
            {
                throw new InvalidOperationException();
            }

On windows phone 7 version if any of this exceptions occur the error message is

InvalidOperationException: Operation is not valid due to the current state of the object.

It would be nice to have some better way to distinguish between User cancel and Error, if some kind of error occurs it would be nice to loop a fixed number of times and try to log in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants