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.

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

[MediapartBridge] Add a bridge for Mediapart #1130

Merged
merged 6 commits into from
May 15, 2019
Merged

[MediapartBridge] Add a bridge for Mediapart #1130

merged 6 commits into from
May 15, 2019

Conversation

jtremesay
Copy link
Contributor

Hi,

I have just created a bridge for the french newspaper Mediapart (https://www.mediapart.fr/).

The bridge has two parameters.
The first one is for viewing articles in a single page. When the parameter is set, the RSS link field redirect to the single page version of the article. By default, Mediapart split long articles on multiple pages and I found this behavior annoying.

The second parameter is for setting the user session cookie and getting the full article in the stream.
Without session cookie:
Without session cookie

With invalid session:
With invalid session cookie

With valid session:
With valid session cookie

@logmanoriginal logmanoriginal merged commit 16bd2ae into RSS-Bridge:master May 15, 2019
@logmanoriginal
Copy link
Member

Looks good to me. Thanks for the PR 👍

Copy link
Contributor Author

@jtremesay jtremesay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just have followed the documentation at https://github.com/RSS-Bridge/rss-bridge/wiki/How-to-create-a-new-Bridge

Example:

<?PHP
// PHP code here
// This line is empty (just imagine it!)

@logmanoriginal
Copy link
Member

Wonder who wrote this?

image

Nevermind...


No problem, I'll fix the Wiki real quick. Thanks for pointing it out 🥇

infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
* If no cookie session is defined, use the default rss stream
* Add a parameter for enabling/disabling the single page mode
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.

None yet

2 participants