-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[DonnonsBridge] Add a new bridge #1441
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a little nitpick.
Apart for that, the code is really nice and will be ready to merge. Sorry for the delay.
Hi @teromene, thanks you for your reply. I added the pattern as asked (see commit c21a304). Donnons.org recently changed their query parameters, so i just fixed what i think is the last error in my bridge with commit 836dc28. I will test my "new version" a few day, and tell you when it's ok to merge my bridge into main project. Have a nice day. |
Hi @teromene, all my tests are OK. You can merge my bridge 👍 |
Looks great to me ! Thank you for the PR and for the thorough testing ! |
This bridge allows french users to scrap pages from https://donnons.org/
I test it since 2 month on my own server and it works nicely.
Before submiting my MR, i checked that my code and commits were following code styling rules.
Any comments or suggestions are very welcome since this is my first "bridge".