Problem/Motivation

The Facebook, Instagram and LinkedIn API integrations are all deprecated and no longer working.

Steps to reproduce

Try to establish an API connection with the current methods.

Proposed resolution

Overhaul logic.

Remaining tasks

Create a new 4.0.x branch
Merge the merge request
Tag a new dev release
Tag a new alpha release

User interface changes

Some elements in the form are changed/moved around.

API changes

Cron logic is removed, this should be handled trough your cron logic, ideally through advanced_cron/ultimate_cron.

Data model changes

Config datamodel changes.

Issue fork social_feed_fetcher-3586333

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

bramdriesen created an issue. See original summary.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Related issues: +#3295588: facebook/graph-sdk is deprecated and doesn't support PHP 8

Adding a related issue for a different social feed module. They are using: "facebook/php-business-sdk": "^24.0"

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

It seems like the Facebook/Instagram and the LinkedIn endpoints which are used in this module are all deprecated. Going to see how easy it is to fix that.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Title: Facebook SDK missing as dependency » Revamp API integrations

Going to make this a broader ticket because there is so many overlap that separating it out will result it patches never getting applied together.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Title: Revamp API integrations » Revamp API integrations (Instagram/Facebook/LinkedIn)

bramdriesen opened merge request !10

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Ideally this should go into a 4.0.x branch because of the amount of changes.
Also happy to become a co-maintainer.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Issue summary: View changes
jorgik’s picture

jorgik commented
Version: 3.0.x-dev » 4.0.x-dev
jorgik’s picture

jorgik commented
Status: Active » Needs review

Created new 4.0.x branch and merged this into it.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Thanks, I'll wait for my LinkedIn API key as I suspect there are some changes required there as well.

bramdriesen opened merge request !11

  • bramdriesen committed df9517f9 on 4.0.x 
    feat: #3586333 Revamp API integrations (Instagram/Facebook/LinkedIn)
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Fixed

Going to close this huuuuge issue and move to smaller more reviewable tasks now. The baseline for code quality is green and I have verified the Facebook and Instagram integration. Will follow up with LinkedIn soon.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.