Skip to content

Feat rewayatfans - #2328

Closed
hamedhani1998 wants to merge 3 commits into
lnreader:masterfrom
hamedhani1998:feat-rewayatfans
Closed

hamedhani1998 wants to merge 3 commits into
lnreader:masterfrom
hamedhani1998:feat-rewayatfans

Conversation

@hamedhani1998

Copy link
Copy Markdown
Contributor

Checklist

  • Update version code if an existing plugin was modified
  • Test changes in Plugin Playground or the app
  • Reference related issues in the PR body (e.g. Closes #xyz)

- WordPress REST API for novel listing and search
- Cover images via _embed
- Chapter ordering by number

@rajarsheechatterjee rajarsheechatterjee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This review was generated by Codex.

Codex found the following structural issues that should be addressed before merging:

  • plugins/arabic/rewayatfans.ts:5-13,123-124: This change removes content from WPPage, but parseChapter still reads arr[0].content.rendered. A strict type-check now reports TS2339 on both accesses. Restore content: { rendered: string } (or use a separate response type that includes it) so the chapter response matches the code that consumes it.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants