search

adonisjs / ally

AdonisJS Social Authentication Provider
MIT License
159 stars 53 forks source link

Backport pull request #91 to 4.1 #106

Closed maxgalbu closed 3 years ago

maxgalbu commented 3 years ago

(these pull requests are done to maintain ally 4.1... someone is still using this probably)

Proposed changes

See pull request #91: Update facebook graph api version

thetutlage commented 3 years ago

Doesn't moving to the new version of Facebook API leads to breaking changes?

maxgalbu commented 3 years ago

There are some deprecated fields, but nothing used by adonisjs:

v4 doesn't seem to change anything interesting for login This is something interesting in graph api v3:

Facebook Login

Access Token Expiration: An access tokens is invalid if the user hasn't engaged the app within the last 90 days. The following default fields have replaced public_profile: id first_name last_name middle_name name name_format picture short_name As a result, the following fields that belonged to public_profile are deprecated: age_range context cover currency devices gender link locale timezone updated_time verified

The rsvp_event and user_managed_groups permissions are deprecated. The user_managed_groups permission can still be used for testing purposes but cannot be submitted for Login Review.

Five new permissions have been added: groups_access_member_info – for receiving member-related data on group content. publish_to_groups – for posting content into a group on behalf of a user. user_age_range – for accessing a person's age range. user_gender – for accessing a person's gender. user_link – for accessing the Facebook profile URL for another user of the app.

thetutlage commented 3 years ago

Thanks a ton for the PR. Will release it a day or so