Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

Feedly provider #58

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Feedly provider #58

wants to merge 6 commits into from

Conversation

calebd
Copy link
Owner

@calebd calebd commented Apr 25, 2014

Add feedly-web provider.

jerson and others added 6 commits April 25, 2014 11:27
@jerson @calebd
optional information
4ba8f8d 
I tried login with test accounts and sometimes users can not have certain fields like username, email or verified

not having these fields cause an error in the application
@jerson @calebd
Update SimpleAuthFaceBookWebProvider.m
b9f382c 
From Facebook: On the web, basic_info is implied with every request and isn't required, although the best practice is to include it. Passing basic_info is required on iOS and Android with every permission request.

A person's public profile refers to the following properties on the user object by default:

id
name
first_name
last_name
link
username
gender
locale
age_range
Other public information

 https://developers.facebook.com/docs/facebook-login/permissions/
@calebd
Cleanup and simplify logic.
85d92c2
@calebd
Added Feedly web configuration
bc913bd 
	* Created web view controller based on simpleauth base
	* Added demo
@calebd
Clean up feedly provider implementation. #49
858e5eb
@calebd
Start work on building omniauth style response. #49
44ec9fc
@calebd calebd mentioned this pull request Apr 25, 2014
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@calebd @jerson