-
Notifications
You must be signed in to change notification settings - Fork 340
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
MTNI-281 ⁃ Store only one page object per URL #17
Comments
# Conflicts: # src/options/base.css # src/options/containers/import/index.jsx # src/options/containers/settings/index.jsx # src/options/layout.jsx # src/options/main.jsx # src/overview/main.jsx
# Conflicts: # src/options/base.css # src/options/containers/import/index.jsx # src/options/containers/settings/index.jsx # src/options/layout.jsx # src/options/main.jsx # src/overview/main.jsx
New thoughts:
Expected behaviour: Later we might also think of adding some more metrics to the decision to keep a freeze dry version in the first place. Maybe something related to how long a person stays on an article, or only when a user choses to store the freeze dry version manually. |
# Conflicts: # src/options/base.css # src/options/containers/import/index.jsx # src/options/containers/settings/index.jsx # src/options/layout.jsx # src/options/main.jsx # src/overview/main.jsx
fixed with #69 |
Gerben provided a nice deduplication framework, where we can flexibly define the rules on when and if to store duplicates.
I may be wrong about that, but It seems to me that we currently won't need to store a new object for every visit, as long as there is no versioning.
It also would make it simpler if we just had one version per page and could index all of them, instead of worrying which versions to index. We may want to store and search multiple versions later though.
Want to back this issue? ?utm_campaign=plugin&utm_content=tracker%2F59103681&utm_medium=issues&utm_source=github Post a bounty on it! We accept bounties via ?utm_campaign=plugin&utm_content=tracker%2F59103681&utm_medium=issues&utm_source=github Bountysource.
The text was updated successfully, but these errors were encountered: