Skip to content

Add ConfigurableApi to support custom providers for Android #111

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

Merged
merged 5 commits into from
Jun 2, 2017
Merged

Add ConfigurableApi to support custom providers for Android #111

merged 5 commits into from
Jun 2, 2017

Conversation

vbuch
Copy link
Contributor

@vbuch vbuch commented May 11, 2017

Since I'm only interested in the Android part, I got an error because I didn't have the ios dir.

@vbuch
Copy link
Contributor Author

vbuch commented May 13, 2017

Added a ConfigurableApi to support custom providers. As long as you provide token and authorize urls, it should work. Only did it for my needs meaning there is not OAuth 1 implementation in here.

@vbuch vbuch changed the title Fix check for ios directory Add ConfigurableApi to support custom providers. May 29, 2017
@vbuch vbuch changed the title Add ConfigurableApi to support custom providers. Add ConfigurableApi to support custom providers for Android May 29, 2017
Valery Buchinsky added 3 commits May 30, 2017 10:48
Also allow configuring the access_token_verb;

refs #109, #102, #64
Also allow configuring the access_token_verb;

refs #109, #102, #64
@arelaxend
Copy link

arelaxend commented Jun 2, 2017

👍 good job!
Please see #117. A ConfigurableApi is great but it can not handle non classical cases such as the Dropbox's one because it needs a "non conventional" parameter to be added in the URL.

@vbuch
Copy link
Contributor Author

vbuch commented Jun 2, 2017

@arelaxend can't you just set this in the configurable urls?

@auser auser merged commit 9de2267 into fullstackreact:master Jun 2, 2017
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