Skip to content

fix(faqwikius): The website moved all content to /novel.#2212

Merged
K1ngfish3r merged 6 commits into
lnreader:masterfrom
MrPanda009:master
May 21, 2026
Merged

fix(faqwikius): The website moved all content to /novel.#2212
K1ngfish3r merged 6 commits into
lnreader:masterfrom
MrPanda009:master

Conversation

@MrPanda009
Copy link
Copy Markdown
Contributor

Checklist

  • [ x ] Update version code if an existing plugin was modified

  • [ x ] Test changes in Plugin Playground or the app

  • [ x ] Reference related issues in the PR body (e.g. Closes #xyz)

  • Changed the site property in the FaqWikiUs plugin to point to the new URL (https://faqwiki.us/novel) and updated the version to 3.0.1 to match the latest site changes.

@K1ngfish3r
Copy link
Copy Markdown
Collaborator

add this line to parseNovel somewhere
loadedCheerio('script').remove();

@MrPanda009
Copy link
Copy Markdown
Contributor Author

Done.

@K1ngfish3r
Copy link
Copy Markdown
Collaborator

Done.

oops, i meant L#60+
should probably rename these parseNovels on top to getNovels or something else later

Comment thread plugins/english/faqwikius.ts Outdated
Comment on lines +57 to +61
async parseNovel(path: string): Promise<Plugin.SourceNovel> {
// Fetches and parses a single novel's full details page (metadata, chapters, status).
// "Details" distinguishes it from the list parser above.
async parseNovelDetails(path: string): Promise<Plugin.SourceNovel> {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

name of this cant be changed, revert

Comment thread plugins/english/faqwikius.ts Outdated
Comment on lines +13 to +15
parseNovels(loadedCheerio: CheerioAPI, searchTerm?: string) {
// Parses a list of NovelItem objects from already-loaded Cheerio HTML.
// "List" clarifies it returns NovelItem[] (used by popularNovels and searchNovels).
parseNovelList(loadedCheerio: CheerioAPI, searchTerm?: string) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lol comments

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was going to sleep after this, my bad.

@K1ngfish3r K1ngfish3r merged commit a502c41 into lnreader:master May 21, 2026
1 check passed
nofzo pushed a commit to nofzo/lnreader-plugins that referenced this pull request May 22, 2026
* fix(faqwikius): update site URL to include '/novel' for correct navigation

* chore: bumped version of faqwiki

* fix(faqwikius): remove script tags from parsed HTML to clean up novel data

* fix(faqwikius): move the remove script tags from parseNovels method to parseNovel method

* chore(faqwikius): rename parseNovels method to parseNovelList and parseNovel to parseNovelDetails for clarity

* fix(faqwikius): reversion of previous commit
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.

2 participants