Replies: 1 comment 1 reply
-
Hi! Thanks for starting the discussion. what was the previous PR @oidebrett ? These are just demo UIs for folks to use and adapt. I don't see an issue with having this in case someone wants this functionality, but would recommend it is easy to remove also if they do not (e.g., if they only want to use NLWeb on their site, they likely would not want a UI to allow scraping and writing to their DB). I would also ensure you have documentation that you will have a write capability to your DB available if you are using this and to use appropriate caution. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi! I've forked NLweb and implemented a UI-based way for users to add sites. Before I open a PR, I wanted to check if this type of contribution is something you'd be open to — just to avoid investing time unnecessarily.
Here’s a quick summary of what I’ve changed:
index.html (in static): Added an expandable left sidebar that lists loaded sites and includes a button to add a new site (via RSS URL or local directory).
fp-chat-interface.js: Added the JavaScript logic for the sidebar.
webserver.py: Introduced two POST API endpoints:
/api/sites/add: Adds a site by RSS URL and name. Validates input and stores documents.
/api/sites/delete: Deletes a site by name.
I raised a previous PR which hasn't been merged yet and seems to have stalled, so just checking if PRs like this are welcomed.
Beta Was this translation helpful? Give feedback.
All reactions