Skip to content

feat(ar/rewayahfans): add روايه فانز plugin#2306

Open
hamedhani1998 wants to merge 1 commit into
lnreader:masterfrom
hamedhani1998:feat-rewayahfans
Open

feat(ar/rewayahfans): add روايه فانز plugin#2306
hamedhani1998 wants to merge 1 commit into
lnreader:masterfrom
hamedhani1998:feat-rewayahfans

Conversation

@hamedhani1998

@hamedhani1998 hamedhani1998 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add new Arabic plugin for rewayahfans.net
  • WordPress REST API for novel listing and search
  • Chapter content via WP REST API with HTML fallback

@K1ngfish3r

K1ngfish3r commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

the standalone plugin is missing... add the plugin here

@K1ngfish3r

K1ngfish3r commented Jul 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
image Picture missing and chapter ordering doesnt seem fine...

parseNovel how-to-be-a-dark-heros-daughter-178

@K1ngfish3r

K1ngfish3r commented Jul 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
image is fetching all the pages required?

normal fetch, popular

hamedhani1998 pushed a commit to hamedhani1998/lnreader-plugins that referenced this pull request Jul 18, 2026
yemen
fix(ar/rewayahfans): add cover images, fix chapter ordering, remove g…
2c9fbdc 
…etAllNovels

- Add getCoverUrls() to batch-fetch featured media URLs via WP REST API
- Fix cover images in popularNovels and searchNovels (was always empty)
- Remove getAllNovels() that fetched ALL pages (very slow)
- Simplify popularNovels to use normal paginated API
- Fix chapter ordering in parseNovel (sort always applied)

Addresses reviewer feedback on PR lnreader#2306
@K1ngfish3r

K1ngfish3r commented Jul 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
image

parseNovel how-to-survive-as-that-villainess-74
image still missing
chapter 62 seems to have snuck in here
summary (if any) is missing
genres (if any) are missing

@K1ngfish3r

K1ngfish3r commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

page 1 and 2 have repeating entries

image image

you can try using adding novel paths to a set, which will record all the paths to ensure no duplicates are pushed

@hamedhani1998
hamedhani1998 force-pushed the feat-rewayahfans branch 2 times, most recently from e6e19c7 to 7af1949 Compare July 21, 2026 07:33
@K1ngfish3r

K1ngfish3r commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

I went to the website
https://rewayahfans.net/%D9%82%D8%A7%D8%A6%D9%85%D8%A9-%D8%A7%D9%84%D8%B1%D9%88%D8%A7%D9%8A%D8%A7%D8%AA/
why isnt this used for getting popularNovels? you can just populate them all in one page no? then use search by filtering them. pagination is not required if it is all in one page

@hamedhani1998

hamedhani1998 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

I went to the website https://rewayahfans.net/%D9%82%D8%A7%D8%A6%D9%85%D8%A9-%D8%A7%D9%84%D8%B1%D9%88%D8%A7%D9%8A%D8%A7%D8%AA/ why isnt this used for getting popularNovels? you can just populate them all in one page no? then use search by filtering them. pagination is not required if it is all in one page

Finsh

@hamedhani1998

hamedhani1998 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

@K1ngfish3r finsh

@K1ngfish3r

K1ngfish3r commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

why merge conflict?
also why public/plugins.json? remove

yemen
feat(ar/rewayahfans): add روايه فانز plugin
9993885 
- WordPress site scraping for novel listing and search
- Cover images from the novels list page
- Chapter listing via REST API with HTML fallback
- parseNovel extracts chapters from novel page directly
- Search filters from cached list
@hamedhani1998

hamedhani1998 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

@K1ngfish3r All done. Merge conflict resolved, public/plugins.json removed. Ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hamedhani1998 @K1ngfish3r