Skip to content
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

Solve potential bibliographic data update issues in database.py #7

Open
mhucka opened this issue Jan 29, 2021 · 1 comment
Open

Solve potential bibliographic data update issues in database.py #7

mhucka opened this issue Jan 29, 2021 · 1 comment
Labels
Enhancement ✨ New feature or request Priority ★ Low priority

Comments

@mhucka
Copy link
Contributor

mhucka commented Jan 29, 2021

Currently, the implementation of Item in database.py includes fields for metadata like title, year, etc. These metadata fields are populated from the LSP/ILS at the time the Item object is created. The fields are used to fill out the /item page. Unfortunately, keeping this data cached introduces the problem that the metadata may be updated in the LSP but never be updated in the Item object kept in the database.

Getting the data dynamically every time /item pages are generated is not an option due to obvious performance reasons, but we should find a way to (e.g.) refresh the data, or otherwise avoid potential data inconsistencies.

@mhucka mhucka added Bug 🐛 Something isn't working Enhancement ✨ New feature or request and removed Bug 🐛 Something isn't working labels Jan 29, 2021
@mhucka mhucka added Bug 🐛 Something isn't working Priority ★★ Medium priority Priority ★ Low priority and removed Enhancement ✨ New feature or request Priority ★★ Medium priority labels Oct 6, 2022
@mhucka
Copy link
Contributor Author

mhucka commented Oct 11, 2022

This is partly helped by having the new admin utility update-item-data. Running it will refresh the data from the LSP.

A problem remains that it's not something that staff can do via the web interface but instead requires staff with CLI access to the server to remember to run every once in a while.

Still, this is no longer really a bug but a question of enhancing the system. Changing this issue from a bug to an enhancement.

@mhucka mhucka added Enhancement ✨ New feature or request and removed Bug 🐛 Something isn't working labels Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ New feature or request Priority ★ Low priority
Projects
None yet
Development

No branches or pull requests

1 participant