Add novelarrow#2239
Conversation
|
why does this close 2237? |
|
please do an interactive git rebase and drop unneeded commits |
"Sorry, my bad. I have another branch to fix the bugs for the NovelBin plugin." |
7ui77
force-pushed
the
add-novelarrow
branch
2 times, most recently
from
99f9e29 to
ea560bc
Compare
Done! |
K1ngfish3r
force-pushed
the
add-novelarrow
branch
from
ea560bc to
954f86d
Compare
K1ngfish3r
left a comment
K1ngfish3r
left a comment
There was a problem hiding this comment.
plugin works well, fix the things and add the icon
| class NovelArrow implements Plugin { | ||
| id = 'novelarrow'; | ||
| name = 'Novel Arrow'; | ||
| icon = 'https://novelarrow.com/favicon-32.png'; |
There was a problem hiding this comment.
add an image to public/static/src/en
| // Required headers to bypass Cloudflare and simulate a mobile browser | ||
| headers = { | ||
| 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8', | ||
| 'User-Agent': 'Mozilla/5.0 (Linux; Android 10; K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.0.0.0 Mobile Safari/537.36', | ||
| 'Referer': 'https://novelarrow.com/', | ||
| 'x-client-platform': 'web-mobile', | ||
| 'x-device-type': 'mobile', | ||
| 'x-version-app': 'web-mobile', | ||
| }; |
There was a problem hiding this comment.
typically you would force a desktop UA, not the other around (considering our main audience is mobile only anyways
Co-authored-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Co-authored-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Co-authored-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Co-authored-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Co-authored-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
|
werent the headers required to bypass cf? why delete? |
I only need the header because I'm using CLI commands to test the code. When pushing to the application, this header isn't needed. |
2 participants
Checklist