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

Make error message more clear #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dracan
Copy link

@dracan dracan commented Feb 22, 2019

The Attempting to use the Advanced interface without first having called ENSessionAdvanced.SetSharedSessionConsumerKey error message didn't also mention the ENSessionAdvanced.SetSharedSessionDeveloperToken method.

I had assumed the Advanced features didn't support developer tokens when seeing this error. Turns out I was accidentally calling ENSession.SetSharedSessionDeveloperToken (ie. the non-advanced version). The fact that the error message missed off the other method lead me down the wrong path.

Resolves #35

The "Attempting to use the Advanced interface without first having called ENSessionAdvanced.SetSharedSessionConsumerKey" didn't also mention the ENSessionAdvanced.SetSharedSessionDeveloperToken method. So I had assumed the Advanced features didn't support developer tokens. Turns out I was accidentally calling ENSession.SetSharedSessionDeveloperToken (the non-advanced version).

Resolves Evernote#20
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.

1 participant