Both sides previous revisionPrevious revisionNext revision | Previous revision |
community_call:minutes20230606 [2023/06/01 16:37] – [1. Development Planning] demiankatz | community_call:minutes20230606 [2023/06/06 13:37] (current) – [1. Development Planning] demiankatz |
---|
====== VuFind® Community Call Minutes: June 6, 2023 ====== | ====== VuFind® Community Call Minutes: June 6, 2023 ====== |
| |
Attending: | Attending: Demian Katz, Claas Kazzer, Stephanie Leary, Ere Maijala, Peter Murray, Andrea Buntz Neiman, Katja Tietze, Susan Turkel, Thomas Wagener |
| |
===== Agenda ===== | ===== Agenda ===== |
=== Pull Request / JIRA Ticket Review (release 9.1) === | === Pull Request / JIRA Ticket Review (release 9.1) === |
| |
The following pull requests and tickets are scheduled for inclusion in release 9.1: | The following pull requests are scheduled for inclusion in release 9.1: |
| |
* [[https://github.com/vufind-org/vufind/pull/2271|#2271]] - Ajax facets - fetch only requested facets from Solr | * [[https://github.com/vufind-org/vufind/pull/2271|#2271]] - Ajax facets - fetch only requested facets from Solr |
* [[https://github.com/vufind-org/vufind/pull/2688|#2688]] - Announce number of search results in title | * [[https://github.com/vufind-org/vufind/pull/2688|#2688]] - Announce number of search results in title |
* Status: awaiting input from PR creator (@stephanieleary) | * Status: awaiting input from PR creator (@stephanieleary) |
* [[https://github.com/vufind-org/vufind/pull/2697|#2697]] - Semantics for saved search table | |
* Status: awaiting final testing | |
* [[https://github.com/vufind-org/vufind/pull/2702|#2702]] - Replace Slick Carousel with accessible-slick | * [[https://github.com/vufind-org/vufind/pull/2702|#2702]] - Replace Slick Carousel with accessible-slick |
* Status: discussion suggests we might want to close this and start over with [[https://splidejs.com/|Splide]] instead. | * Status: Discussion suggests we might want to close this and start over with [[https://splidejs.com/|Splide]] instead (Stephanie agrees). Ere investigated whether there was reusable code from finna, but it appears their usage of Splide was in a different context. Chris has volunteered to begin the investigation. |
* [[https://github.com/vufind-org/vufind/pull/2723|#2723]] - Component: confirmation menu | * [[https://github.com/vufind-org/vufind/pull/2723|#2723]] - Component: confirmation menu |
* Status: needs significant work; might be reprioritized to 10.0 | * Status: needs significant work; might be reprioritized to 10.0. Chris suggested that we might be able to use the newly introduced [[https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/popover|popover property]] ([[https://html.spec.whatwg.org/multipage/popover.html|spec]]) in combination with a polyfill to address this in a standards-based way. |
* [[https://github.com/vufind-org/vufind/pull/2763|#2763]] - SetupThemeResources: Ensure 'Content-Type' is never escaped | * [[https://github.com/vufind-org/vufind/pull/2763|#2763]] - SetupThemeResources: Ensure 'Content-Type' is never escaped |
* Status: awaiting input from PR creator (@mtrojan-ub) | * Status: awaiting input from PR creator (@mtrojan-ub) |
* Status: requires further discussion | * Status: requires further discussion |
* [[https://github.com/vufind-org/vufind/pull/2852|#2852]] - Add header and h2 for title and author | * [[https://github.com/vufind-org/vufind/pull/2852|#2852]] - Add header and h2 for title and author |
* Status: awaiting final consensus about approach; requires significant conflict resolution | * Status: awaiting final consensus about approach; requires significant conflict resolution. Chris is okay with merging this as long as other accessibility experts agree that it is useful -- he raised some questions about the necessity of the extra header tags but does not feel that this makes things worse. |
* [[https://github.com/vufind-org/vufind/pull/2857|#2857]] - Add support for "uncertain" item availability status | |
* Status: in review | |
* [[https://github.com/vufind-org/vufind/pull/2860|#2860]] - Add LibGuidesAZ backend for Databases A-Z list | * [[https://github.com/vufind-org/vufind/pull/2860|#2860]] - Add LibGuidesAZ backend for Databases A-Z list |
* Status: in review | * Status: in review |
| * [[https://github.com/vufind-org/vufind/pull/2874|#2874]] - Improve accessibility for checkboxes |
| * Status: in review; probably needs to be broken into smaller parts for manageability |
* [[https://github.com/vufind-org/vufind/pull/2914|#2914]] - Turn my account menu into ul, because it is a list | * [[https://github.com/vufind-org/vufind/pull/2914|#2914]] - Turn my account menu into ul, because it is a list |
* Status: in review | * Status: in review; Chris raised the question of whether a menu tag would be a better option than a ul, but Stephanie offered some alternative suggestions on the PR. |
| * [[https://github.com/vufind-org/vufind/pull/2922|#2922]] / [[https://github.com/vufind-org/vufind/pull/2924|#2924]] - Correct SCSS compile extend error |
| * Status: #2922 is currently in review; #2924 adds some continuous integration functionality that should be merged after #2922 is fixed. |
| |
| The following JIRA tickets represent features, fixes and improvements that need to be started: |
| |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1252|VUFIND-1252]] - Add ID prefix option to deletes script |
| * Status: assigned to Demian; not started. |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1568|VUFIND-1568]] - Make Solr port configurable during installation |
| * Status: assigned to Demian; not started. |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1598|VUFIND-1598]] - Allow users to view their proxy relationships |
| * Status: unassigned; Ere mentioned that this is done locally for Koha. |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1599|VUFIND-1599]] - FOLIO: improve documentation/error handling around permissions |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1604|VUFIND-1604]] - Templates: components do not work inside of slots |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1606|VUFIND-1606]] - Upgrade vanilla-cookieconsent to fix HTML validation when new version available |
| * Status: Assigned to Ere; in progress. |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1607|VUFIND-1607]] - Embedded records (in results list) have invalid aria and/or role attributes |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1611|VUFIND-1611]] - Accessible local CAPTCHA option |
| * Status: unassigned, but Chris is doing some research on options. |
| |
| The following JIRA tickets represent test suite improvement ideas: |
| |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1351|VUFIND-1351]] - Create Mink tests for email verification functionality |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1471|VUFIND-1471]] - Create mink test for excluding hierarchical facets |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1541|VUFIND-1541]] - Add Mink test coverage for Markdown rendering |
| * Status: assigned to Chris; not started. |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1585|VUFIND-1585]] - Create Mink tests for library card functionality |
| * Status: unassigned |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1603|VUFIND-1603]] - Add test to verify that search tabs are created properly |
| * Status: assigned to Ere; not started. |
| |
| The following JIRA tickets represent tasks to be handled closer to the final release: |
| |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1608|VUFIND-1608]] - Update language translation files for release 9.1 |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1609|VUFIND-1609]] - Update dependencies for release 9.1 |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1612|VUFIND-1612]] - Double check for "PHP version 7" in file comments |
| * [[https://openlibraryfoundation.atlassian.net/browse/VUFIND-1613|VUFIND-1613]] - Update GitHub branch protection rules for release-* branches |
| |
=== Scheduling Release 9.0.2 === | === Scheduling Release 9.0.2 === |
==== 2. SolrMarc Sustainability ==== | ==== 2. SolrMarc Sustainability ==== |
| |
Demian has been talking with Bob Haschart, the lead developer of SolrMarc, about improving the sustainability of the project, since it has not seen a new release in some time. Bob is receptive to expanding involvement in SolrMarc; if you are interested in being more involved in that project or in helping to review open issues and pull requests, please let Demian know. | Demian has been talking with Bob Haschart, the lead developer of SolrMarc, about improving the sustainability of the project. These discussions led to a new release incorporating some recent bug fixes, and this release has been incorporated into the dev branch already. Bob is receptive to expanding involvement in SolrMarc; if you are interested in being more involved in that project or in helping to review open issues and pull requests, please let Demian know. Peter asked about skill requirements: mainly Java, MARC and Solr familiarity. |
| |
==== 3. Next Meeting Date ==== | ==== 3. Next Meeting Date ==== |
==== 4. Open Q&A / Other Topics? ==== | ==== 4. Open Q&A / Other Topics? ==== |
| |
None this month. | Reminder: WOLFcon and the Summit are coming up in August! |
| |
| Ere will be out of office for July. |
| |
| Peter reported that a University of Colorado accessibility review revealed some problems with the autocomplete/autosuggest functionality. He demonstrated the problem with NVDA. Stephanie is going to compare notes with Peter as she is also working on a review; she suspects it's aria-related. Claas and Ere will also look into this locally. Demian has shared the results with Chris so that he can investigate further. Peter will open a JIRA ticket. |
| |
| Demian mentioned that the dev-10.0 branch has been established to merge PRs that are too advanced for 9.1. |
| |
| Demian also mentioned that he created an accessibility PR tag in GitHub to help keep track of accessibility-related PRs. |
===== Next Call ===== | ===== Next Call ===== |
| |