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
ContributorAuthor

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 threadplugins/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 threadplugins/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
ContributorAuthor

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:masterMay 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
@github-actionsgithub-actionsBot locked as resolved and limited conversation to collaborators Jul 22, 2026
Sign up for freeto 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

@MrPanda009@K1ngfish3r