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.

Sign up for GitHub

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

Jump to bottom

Add news module integration #10

Open
theDyingMountain opened this issue Dec 5, 2018 · 3 comments
Open

Add news module integration #10

theDyingMountain opened this issue Dec 5, 2018 · 3 comments
Labels
feature

Comments

@theDyingMountain
Copy link

Many users are familiar with the contao news module. It would be nice to add the possibility integrating posts in the news module. Then you can mix "normal" contao news with facebook news.

Note: I had done this functionality in my own contao facebook connector, but don't want to maintain this anymore. Your bundle is smarter and well-designed 😉
@m-vo
Copy link
Owner

m-vo commented Dec 21, 2018

How would you suggest integrating the news? I don't think adding the news to the Contao repositories would be a good idea because this way you'd need to synchronize both sources (and the data path FB -> Contao is unidirectional). But maybe somehow allowing the reader to access the FB news repositories?

@m-vo m-vo added the feature label Dec 21, 2018
@theDyingMountain
Copy link
Author

I don't think it's necessary to synchronize both sources. You can offer an option before the synchronization. If the contao news modul option is selected, you can add entries in the tl_news table and add a flag to mark the news as facebook posts. Then you can mark all flagged news as readonly in the contao backend. The main advantage is an improved clarity and easier maintenance for editors.

But i agree that allowing the reader access to the FB news repositories will be an simpler and most likely faster option.

@bastlwastl
Copy link

For example, this was implemented in the following module for Contao 3.
https://github.com/postyou/Contao-Aggregator

trcwebdesign added a commit to trcwebdesign/contao-facebook-import that referenced this issue May 13, 2020
@trcwebdesign
Raise GRAPH_API_VERSION to 7.0 for avoiding FB errors
251905f 
Facebook needs to queried by v7.0 for the page-id/posts and page-id/events endpoints. 
Older versions will display an error like this
error": {
    "message": "(m-vo#10) This endpoint requires the 'manage_pages' or 'pages_read_engagement' permission or the 'Page Public Content Access' feature. Refer to https://developers.facebook.com/docs/apps/review/login-permissions#manage-pages and https://developers.facebook.com/docs/apps/review/feature#reference-PAGES_ACCESS for details.",
    "type": "OAuthException",
    "code": 10,
@trcwebdesign trcwebdesign mentioned this issue May 13, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bastlwastl @theDyingMountain @m-vo