-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Remove index.json and index.html generation #502
Conversation
Hmm this PR seems fine but I'd like to make the upgrade UX clearer. Can we rename any
That should make it clear that users need to manually export somehow in order to update those files. |
@pirate I renamed the indexes if they are present. Anything else missing? |
I liked the |
@kedorlaomer you can still generate it using the |
Summary
At the end of the
cli
commands (add, remove, etc) the json and html indexes were still being generated. This is a slow process dependent on the number of archived links (size of the archive). This PR removes that generation, as they can be generated on demand using thearchivebox list
command.**Related issues: #461
Changes these areas