-
Notifications
You must be signed in to change notification settings - Fork 8
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
Consider caching isbn definitions on load #5
Comments
maybe as a postinstall script or something of this kind? |
I was thinking more of making the request as part of an init sort of thing. So the request is made once, when the application is started. |
I wouldn't like it to be the default behaviour as that would:
I tried to improve the documentation on how one could update the groups without having to rely on those being updated by the module maintainers, could that be enough? Edit: I reverted the change in the documentation as that's actually bad advice: as re-running npm install in the root project will revert the group update |
I added a CI action to auto-update groups data every 1st of the month: today was the first auto-update+patch version publication triggered that way. This means that as long as this action is up and there is no breaking change in the isbn-international.org API, we [the maintainers] could well totally forget about this repo and it would still be periodically updated. |
There has been change Dec 18, 2020. Is auto update working? (Top margin in pdf says so https://www.isbn-international.org/range_file_generation) |
The action was disabled for some reason I don't get, but now it's reactivated, and I manually ran (8afee59) and published ( |
The ISBN definitions seem to be updated fairly frequently as far as I can tell. Would it be possible to make a request on loading the library for the new definitions? The hard-coded ones could be used as back-up if the library is offline / the website is down. If that seems like something that we want to do I could make a PR.
The text was updated successfully, but these errors were encountered: