Attending: Leila Gonzales, Chris Hallberg, Demian Katz, Eoin Kilfeather, Jochen Lienhard, Conor Sheehan
Demian has begun work on revising the service manager configuration in PR #1067. He was originally thinking of merging the PR sooner, but the discovery of some bugs revealed that fixing the configuration as discussed last time is also a good way to thoroughly test everything.
The checklist on the PR has been updated to reflect outstanding tasks, and Demian (with Chris' help) will continue working through it as time permits. Some other work is more or less on hold until this is completed.
It appears that SolrMarc 3.1 has been released; Demian will upgrade VuFind when time permits, which will allow further progress on this PR.
Suggestions for next priorities from the roadmap are encouraged.
No news.
Chris wanted to explain a couple of things:
- The new version of FontAwesome (5) is out. We use this library for our icons. Chris was disappointed by the compatibility of the new version; some icons have broken backward compatibility, and some new features of the new version don't work with VuFind. Right now, it looks like a lot of work for very little pay-off to upgrade, so this won't be happening too soon.
- Chris is also planning on refactoring the dynamic cover image generator soon.
Tod found some problems related to old .jar files bundled with the browse handler project; he has resolved this (and ensured that the Travis build still passes).
No news.
Leila is working on some geographic improvements – PR #1086 is the first step toward migrating from OpenLayers to Leaflet by making base maps configurable. PR #1087 allows better logging during indexing of coordinates to help with troubleshooting. These are currently awaiting Demian's review.
Leila has two more pull requests lined up to open after these are merged (character encoding indexing fixes; Leaflet conversion). Leaflet offers better clustering than OpenLayers. She will also update the wiki after merging.
There was a question about whether there is a simple way to inject a configuration into a class without creating a factory. The short answer: it's probably best to use a factory given current VuFind conventions, but the new ServiceManager v3 refactor may make it easier to create a shared factory that injects configurations in a specific way, if such a pattern is useful, in order to avoid redundancy.
There was also a discussion of the getRealTimeHoldings call in the holdings tab code, during which Demian may have misspoken about its purpose and history. You shouldn't always trust him first thing in the morning.
The next call will be Tuesday, January 30, 2018 at 9am Eastern Standard Time (14:00 GMT).