Skip to content

NovelNice - #2395

Merged
rajarsheechatterjee merged 2 commits into
lnreader:masterfrom
52191314:feat/novelnice
Aug 8, 2026
Merged

NovelNice#2395
rajarsheechatterjee merged 2 commits into
lnreader:masterfrom
52191314:feat/novelnice

Conversation

@52191314

@52191314 52191314 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Fixes #2369

BoxNovel rebranded to Novel Nice (novelnice.com). Added as a new madara plugin (id novelnice, v2.2.0) — the existing boxnovel entry is left untouched, so current installs keep working and the rebrand installs fresh.

  • sources.json: new novelnice entry (same site, useNewChapterEndpoint)
  • filters/novelnice.json + icon: copied from boxnovel (same genre list)
  • template.ts: fetch all pages of paginated chapter lists (some sites cap the ajax response at 200 chapters and paginate via ?t=N links)

If blocked by Cloudflare, open the site in the app webview and solve the challenge once — the cf_clearance cookie then makes search work for its TTL.

BoxNovel rebranded to Novel Nice (novelnice.com). Add it as a new
madara instance (id novelnice, v2.2.0) instead of touching the existing
boxnovel entry, so existing installs keep working and the rebrand
installs fresh.

- sources.json: new novelnice entry (same site, useNewChapterEndpoint)
- filters/novelnice.json + icon: copied from boxnovel (same genre list)

Fixes lnreader#2369
@raziel100

Copy link
Copy Markdown

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

@52191314

52191314 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.

EDIT: The current one still only shows the first page of chapters.

@52191314 52191314 closed this Aug 8, 2026
@52191314 52191314 reopened this Aug 8, 2026
@raziel100

Copy link
Copy Markdown

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.

EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

Some sites cap the ajax/chapters response at 200 entries and paginate
via ?t=N links (data-page). Detect the pagination links on the first
page and fetch the remaining pages, concatenating the HTML before
parsing, so the full chapter list is returned instead of the cap.
@52191314

52191314 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

The template.ts fix was uncommitted when you reviewed. Initially, the template was updated to send the user to a 1.0.0 Novel Nice instead of Boxnovel 2.2.3. But I undid.

@raziel100

Copy link
Copy Markdown

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

The template.ts fix was uncommitted when you reviewed. Initially, the template was updated to send the user to a 1.0.0 Novel Nice instead of Boxnovel 2.2.3. But I undid.

I have noticed that, so what the changes in template fixes? Only the chapter list?

@52191314

52191314 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

Is there any reason why this pr was created? The boxnovel plug-in already redirects to novelnice.

Ahhhh. Saw the issue was still open. My bad.
EDIT: The current one still only shows the first page of chapters.

Only for a few novels, most of them show all chapters, this pr shouldn't fix that particular issue, is literally the same thing, you just changed boxnovel do novelnice. Nothing in the code or script has been changed.

The template.ts fix was uncommitted when you reviewed. Initially, the template was updated to send the user to a 1.0.0 Novel Nice instead of Boxnovel 2.2.3. But I undid.

I have noticed that, so what the changes in template fixes? Only the chapter list?

Yes, that's right.

@rajarsheechatterjee
rajarsheechatterjee merged commit b3b99c1 into lnreader:master Aug 8, 2026
2 checks passed
@52191314
52191314 deleted the feat/novelnice branch August 9, 2026 06:52
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.

Someone please fix box novel.

3 participants