Skip to content

Add search for novelfire#3009

Merged
dipu-bd merged 2 commits into
lncrawl:devfrom
GabrielCWT:feat/add-search-novelfire
Jun 2, 2026
Merged

Add search for novelfire#3009
dipu-bd merged 2 commits into
lncrawl:devfrom
GabrielCWT:feat/add-search-novelfire

Conversation

@GabrielCWT

Copy link
Copy Markdown
Contributor

No description provided.

@GabrielCWT

Copy link
Copy Markdown
Contributor Author

Unfortunately I am unsure how to go about testing this feature.

I have ran make index-gen to update the index and then ran uv run python3 -m lncrawl search "Barbarian" -s "novelfire"

However, it still errors with NotImplementedError(). Would love to get some help on this!

@dipu-bd

dipu-bd commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

It needs index generation. If I merge this, it should start working.

search feature is not yet supported with the new architecture properly. once other priority features are done, it will be picked up.

@dipu-bd dipu-bd merged commit 1a5ad43 into lncrawl:dev Jun 2, 2026
6 checks passed
dipu-bd pushed a commit that referenced this pull request Jun 12, 2026
* Add search for novelfire

* Fix linting
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