Warning: This page has not been updated in over over a year and may be outdated or deprecated.
community_call:minutes20240507
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revision | ||
community_call:minutes20240507 [2024/04/30 17:53] – demiankatz | community_call:minutes20240507 [2024/05/07 15:57] (current) – demiankatz | ||
---|---|---|---|
Line 1: | Line 1: | ||
====== VuFind® Community Call Minutes: May 7, 2024 ====== | ====== VuFind® Community Call Minutes: May 7, 2024 ====== | ||
- | Attending: | + | Attending: |
+ | Apologies: Ere Maijala | ||
===== Agenda ===== | ===== Agenda ===== | ||
Line 15: | Line 16: | ||
* [[https:// | * [[https:// | ||
* Status: We need to determine if this can be fixed in 9.1.2 or needs to be delayed. | * Status: We need to determine if this can be fixed in 9.1.2 or needs to be delayed. | ||
+ | * :!: ACTION ITEM: Demian and Ere should discuss | ||
* [[https:// | * [[https:// | ||
* Status: Needs review. | * Status: Needs review. | ||
+ | * :!: ACTION ITEM: please review! | ||
+ | * [[https:// | ||
+ | * Status: Needs review. | ||
+ | * :!: ACTION ITEM: please review! | ||
=== Pull Request Review (release 10.0) === | === Pull Request Review (release 10.0) === | ||
Line 25: | Line 31: | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
* :!: ACTION ITEM: Chris should determine next steps. | * :!: ACTION ITEM: Chris should determine next steps. | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
+ | * :!: ACTION ITEM: Change milestone. | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
* :!: ACTION ITEM: Chris should try to fix failing tests. | * :!: ACTION ITEM: Chris should try to fix failing tests. | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
* :!: ACTION ITEM: Chris should look at style/ | * :!: ACTION ITEM: Chris should look at style/ | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
- | * :!: ACTION ITEM: Chris should | + | * :!: ACTION ITEM: Chris will continue work; others should test/review. |
* [[https:// | * [[https:// | ||
* Status: In progress | * Status: In progress | ||
Line 45: | Line 52: | ||
* [[https:// | * [[https:// | ||
- | * Status: Currently in review. | + | * Status: Currently in review; likely to get deferred to 10.1. |
* :!: ACTION ITEM: Johannes should continue working on review comments. | * :!: ACTION ITEM: Johannes should continue working on review comments. | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
* :!: ACTION ITEM: Chris should review the latest changes. | * :!: ACTION ITEM: Chris should review the latest changes. | ||
+ | * [[https:// | ||
+ | * Status: In review | ||
+ | * :!: ACTION ITEM: Review, please! | ||
+ | * [[https:// | ||
+ | * Status: This new feature was introduced in response to widespread problems with abusive crawling; even though it is new, we should aim to get it into the release to support users facing these challenges. (Mario mentioned that his team is evaluating mod_security; | ||
+ | * :!: ACTION ITEM: Demian and Ere will work through the review process; input from others would be appreciated. | ||
== Smaller Features/ | == Smaller Features/ | ||
- | * [[https:// | ||
- | * Status: In review. | ||
- | * :!: ACTION ITEM: Demian should review latest round of changes. | ||
* [[https:// | * [[https:// | ||
* Status: In review. | * Status: In review. | ||
Line 62: | Line 72: | ||
* Status: In review | * Status: In review | ||
* :!: ACTION ITEM: Waiting for progress from @sturkel89 | * :!: ACTION ITEM: Waiting for progress from @sturkel89 | ||
+ | * [[https:// | ||
+ | * Status: Needs review | ||
+ | * :!: ACTION ITEM: please review | ||
+ | * [[https:// | ||
+ | * Status: In review | ||
+ | * :!: ACTION ITEM: Demian will act on Maccabee' | ||
== Specialized PRs Requiring Input from Creators == | == Specialized PRs Requiring Input from Creators == | ||
Line 67: | Line 83: | ||
* [[https:// | * [[https:// | ||
* Status: Under discussion; will likely get postponed to a future release unless more input is received. | * Status: Under discussion; will likely get postponed to a future release unless more input is received. | ||
+ | * :!: ACTION ITEM: move to 10.1 milestone due to inactivity | ||
* [[https:// | * [[https:// | ||
* Status: On hold awaiting more input from the Google Scholar team | * Status: On hold awaiting more input from the Google Scholar team | ||
- | * :!: ACTION ITEM: @mtrojan-ub will update when he hears back, or close if no response is received. | + | * :!: ACTION ITEM: @mtrojan-ub will close |
* [[https:// | * [[https:// | ||
* Status: Under discussion. | * Status: Under discussion. | ||
- | * :!: ACTION ITEM: @mtrojan-ub | + | * :!: ACTION ITEM: Demian |
* [[https:// | * [[https:// | ||
* Status: In progress. | * Status: In progress. | ||
+ | * :!: ACTION ITEM: @robertlange81 should continue work (expected to happen this week) | ||
=== Scheduled JIRA Tickets (release 10.0) === | === Scheduled JIRA Tickets (release 10.0) === | ||
Line 81: | Line 99: | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
- | * [[https:// | + | * :!: ACTION ITEM: move to 10.1 milestone |
- | * Status: Needs more investigation. Can we refactor | + | |
== Smaller Fixes/ | == Smaller Fixes/ | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: Demian |
+ | * :!: ACTION ITEM: Ere should review Demian' | ||
* [[https:// | * [[https:// | ||
* Status: This is an unavoidable problem in Bootstrap 3; we should investigate whether Bootstrap 5 improves the situation. | * Status: This is an unavoidable problem in Bootstrap 3; we should investigate whether Bootstrap 5 improves the situation. | ||
+ | * :!: ACTION ITEM: check Bootstrap 5 situation; decide on next steps | ||
* [[https:// | * [[https:// | ||
* Status: Maccabee and Demian will look into if/when time permits. | * Status: Maccabee and Demian will look into if/when time permits. | ||
Line 96: | Line 115: | ||
* Status: Chris should investigate. | * Status: Chris should investigate. | ||
* [[https:// | * [[https:// | ||
- | * Status: | + | * Status: |
== Test Suite Improvements == | == Test Suite Improvements == | ||
Line 111: | Line 130: | ||
* [[https:// | * [[https:// | ||
+ | * Status: this was done recently, but we will double-check closer to the release date. | ||
* [[https:// | * [[https:// | ||
+ | * Status: will do after releasing 9.1.2 | ||
+ | * :!: ACTION ITEM: set release date for 9.1.2 first | ||
* [[https:// | * [[https:// | ||
+ | * Status: will begin about a month before the release date. | ||
+ | * :!: ACTION ITEM: Demian will begin this process on May 20th. | ||
==== 2. Technical Discussion: Theme Development Next Steps / Collaboration ==== | ==== 2. Technical Discussion: Theme Development Next Steps / Collaboration ==== | ||
+ | Ere believes that the Bootstrap5 theme is nearly done. Some accessibility testing is still required, but he believes that it will be more accessible than the bootstrap3 theme even as it is. He has written a [[development: | ||
+ | |||
+ | As noted above, Ere suggested deferring the replacement of Grunt scripts with NPM scripts until 11.0, since rewriting the current scripts required by bootstrap3 will be difficult, and some will no longer be needed once bootstrap3 is removed. | ||
+ | |||
+ | We should also document the theme transition plan in the changelog: beta bootstrap5 in 10.0, mature bootstrap5 and deprecated bootstrap3 in 10.1, removal of bootstrap3 in 11.0. | ||
+ | |||
+ | Susan raised the question of how sandal vs. sandal5 naming will be addressed during transition; we still need to work out some details there. Right now, the sandal5 theme exists to enable parallel testing of the bootstrap3-based sandal and the bootstrap5-based sandal5. When we get to 10.1, we may need to change the default to sandal5. The question then becomes, do we rename sandal5 to sandal in 11.0, or keep the numbered name? Is there another way forward? We should discuss as things continue to evolve. | ||
==== 3. Technical Discussion: Database Code Migration Strategy ==== | ==== 3. Technical Discussion: Database Code Migration Strategy ==== | ||
+ | |||
+ | Demian has made significant progress on introducing database services and entity interfaces, but there is more to do. Any help, particularly with review so that work does not stall, would be greatly appreciated. | ||
==== 4. Future of Slack ==== | ==== 4. Future of Slack ==== | ||
+ | No updates yet from the OLF side; it's taking time to " | ||
==== 5. Open Q&A / Other Topics? ==== | ==== 5. Open Q&A / Other Topics? ==== | ||
+ | None this month. | ||
===== Next Call ===== | ===== Next Call ===== | ||
community_call/minutes20240507.txt · Last modified: 2024/05/07 15:57 by demiankatz