About Features Downloads Getting Started Documentation Events Support GitHub

Love VuFind®? Consider becoming a financial supporter. Your support helps build a better VuFind®!

Site Tools


Warning: This page has not been updated in over over a year and may be outdated or deprecated.
developers_call:minutes20160412

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision
Previous revision
developers_call:minutes20160412 [2016/04/11 18:16] – [1. Development Updates] demiankatzdevelopers_call:minutes20160412 [2016/04/21 12:56] (current) – [4. Other Topics?] demiankatz
Line 1: Line 1:
 ====== VuFind Developers Call Minutes: April 12, 2016 ====== ====== VuFind Developers Call Minutes: April 12, 2016 ======
  
-Attending: +Attending: Matthias Edel, Leila Gonzales, Chris Hallberg, Demian Katz, André Lahmann, Jochen Lienhard, Brad Patton, Jay Roos
  
 ===== Agenda ===== ===== Agenda =====
Line 9: Line 9:
 === JIRA Tickets === === JIRA Tickets ===
  
-  * [[https://vufind.org/jira/browse/VUFIND-1160|VUFIND-1160]] - This ticket reports a side effect of persistent hidden filters that negatively impacted the Solr-driven new items module; a fix has been committed.+  * [[https://vufind.org/jira/browse/VUFIND-1160|VUFIND-1160]] - This ticket reports a side effect of persistent hidden filters that negatively impacted the Solr-driven new items module; a fix has been committed (PR [[https://github.com/vufind-org/vufind/pull/672|#672]]).
   * [[https://vufind.org/jira/browse/VUFIND-1161|VUFIND-1161]] - This ticket documents a PrimoCentral problem (no VuFind workaround yet; waiting for feedback from Ex Libris).   * [[https://vufind.org/jira/browse/VUFIND-1161|VUFIND-1161]] - This ticket documents a PrimoCentral problem (no VuFind workaround yet; waiting for feedback from Ex Libris).
   * [[https://vufind.org/jira/browse/VUFIND-1162|VUFIND-1162]] - This ticket describes a lightbox-related bug in master; it has been fixed.   * [[https://vufind.org/jira/browse/VUFIND-1162|VUFIND-1162]] - This ticket describes a lightbox-related bug in master; it has been fixed.
Line 17: Line 17:
 === Pull Requests === === Pull Requests ===
  
-  * [[https://github.com/vufind-org/vufind/pull/653|#653]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/653|#653]], [[https://github.com/vufind-org/vufind/pull/657|#657]] - merged - Voyager ILS driver fixes. 
-  * [[https://github.com/vufind-org/vufind/pull/654|#654]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/654|#654]], [[https://github.com/vufind-org/vufind/pull/669|#669]], [[https://github.com/vufind-org/vufind/pull/675|#675]] - merged - Improved lightbox functionality. 
-  * [[https://github.com/vufind-org/vufind/pull/655|#655]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/655|#655]] - merged - Improved mechanism for initialization of Javascript. 
-  * [[https://github.com/vufind-org/vufind/pull/656|#656]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/656|#656]][[https://github.com/vufind-org/vufind/pull/667|#667]] - merged - Translation updates. 
-  * [[https://github.com/vufind-org/vufind/pull/657|#657]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/658|#658]] - merged - Adjustments to the display of pickup locations when placing and displaying holds.
-  * [[https://github.com/vufind-org/vufind/pull/658|#658]] - merged -+
   * [[https://github.com/vufind-org/vufind/pull/659|#659]] - in progress - Adds a Gruntfile to automate Node-related tasks; opened for discussion purposes.   * [[https://github.com/vufind-org/vufind/pull/659|#659]] - in progress - Adds a Gruntfile to automate Node-related tasks; opened for discussion purposes.
-  * [[https://github.com/vufind-org/vufind/pull/660|#660]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/660|#660]] - merged - Minor bootstrap3 theme fixes. 
-  * [[https://github.com/vufind-org/vufind/pull/661|#661]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/661|#661]] - merged - Minor fix to DynamicRoleProvider. 
-  * [[https://github.com/vufind-org/vufind/pull/662|#662]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/662|#662]] - merged - Reverted broken Primo Central batch loading logic. 
-  * [[https://github.com/vufind-org/vufind/pull/663|#663]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/663|#663]] - merged - Made advanced search URLs more tolerant of missing parameters (default to AND operator). 
-  * [[https://github.com/vufind-org/vufind/pull/664|#664]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/664|#664]] - merged - Truncate long values upon insert into resource table for better compatibility with strict length checks in database. 
-  * [[https://github.com/vufind-org/vufind/pull/665|#665]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/665|#665]] - merged - Primo Central bug fix. 
-  * [[https://github.com/vufind-org/vufind/pull/666|#666]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/666|#666]] - merged - Improved handling of ILS exceptions (for better user feedback). 
-  * [[https://github.com/vufind-org/vufind/pull/667|#667]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/668|#668]] - merged - New optional feature: jump directly to record when only one match found in search results. 
-  * [[https://github.com/vufind-org/vufind/pull/668|#668]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/670|#670]] - merged - Add hostname to detailed exception messages (to help troubleshooting in load-balanced scenarios). 
-  * [[https://github.com/vufind-org/vufind/pull/669|#669]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/671|#671]] - merged - Improvements to Aleph ILS driver. 
-  * [[https://github.com/vufind-org/vufind/pull/670|#670]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/672|#672]] - merged - Fix for [[https://vufind.org/jira/browse/VUFIND-1160|VUFIND-1160]] (described above). 
-  * [[https://github.com/vufind-org/vufind/pull/671|#671]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/673|#673]] - merged - Fix for [[https://vufind.org/jira/browse/VUFIND-1163|VUFIND-1163]] (described above). 
-  * [[https://github.com/vufind-org/vufind/pull/672|#672]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/674|#674]] - merged - Broader support for case-sensitive Unix locale names. 
-  * [[https://github.com/vufind-org/vufind/pull/673|#673]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/676|#676]] - merged - Fixes/enhancements for XCNCIP2 ILS driver. 
-  * [[https://github.com/vufind-org/vufind/pull/674|#674]] - merged - +  * [[https://github.com/vufind-org/vufind/pull/677|#677]] - merged - Syntax fixes for searchspecs.yaml (for compatibility with newer, stricter YAML parsing library).
-  * [[https://github.com/vufind-org/vufind/pull/675|#675]] - merged - +
-  * [[https://github.com/vufind-org/vufind/pull/676|#676]] - merged -+
  
 === Code Freeze === === Code Freeze ===
Line 49: Line 46:
  
 === Javascript Reorganization === === Javascript Reorganization ===
 +
 +All major work is now done, so we will drop this item from the agenda. However, we should continue to discuss front end improvements on a regular basis.
 +
 +Things to do:
 +
 +  - Discuss Grunt
 +  - Finish "bootstrap3-list" pull request
 +  - Fix the "two search boxes for responsiveness" problem
 +  - Settle on code styling (see [[https://github.com/airbnb/javascript/tree/master/es5|suggested reading]])
 +
 +André would like to do more work with Grunt and hopes to look into it in a month or so. Work on updating the Foundation5 theme is underway; the branch will be updated to match release-3.0 in the near future.
  
 === API === === API ===
 +
 +No news.
  
 === Modularization === === Modularization ===
 +
 +No news.
  
 === Next ZF Version === === Next ZF Version ===
 +
 +No news.
  
 === Improved Use of Permissions === === Improved Use of Permissions ===
 +
 +No news.
  
 === Improved Geographic Features === === Improved Geographic Features ===
  
-==== 4. Other Topics? ====+Some discussion by email after last call. Next steps: add geographic fields to Solr schema and standardize map display to use OpenLayers instead of Google Maps. Leila will begin working on some of this soon (new geographic project with a 3.0 base) and will probably have more to say next time. 
 +==== 3. Other Topics? ==== 
 + 
 +=== Git PR Tagging === 
 + 
 +Chris brought up the idea of standardizing our pull request labels so we can better keep track of when things are expected to be merged, who is supposed to be taking next steps, etc. 
 + 
 +Starting point: bug fix vs. new feature vs. discussion. 
 + 
 +One common pattern: tagging based on component (e.g. front-end vs. back-end) or technology (PHP vs. Javascript). 
 + 
 +One specific scenario: PR is submitted, but submitter never follows up on Demian's feedback, and Demian doesn't have time to fix it himself. Chris will look for a good precedent for naming for a tag for this situation. 
 + 
 +After looking at the [[https://github.com/twbs/bootstrap/pulls?page=2&q=is%3Apr+is%3Aopen|Bootstrap repository]], Demian and Chris have proposed the following [[https://github.com/vufind-org/vufind/labels|labels]] and [[https://github.com/vufind-org/vufind/pulls|applied them accordingly]]: 
 + 
 +  * **awaiting reply** - The VuFind dev team eagerly await the submitter's feedback. 
 +  * **breaking** - This PR contains code that will likely cause some backwards compatibility problems. 
 +  * **bugfix** - This branch deals with fixing a found bug. 
 +  * **discussion** - Your opinion would be greatly appreciated! 
 +  * **improvement** - Code focused on improving a currently existing feature. 
 +  * **new feature** - Code focused on exploring new areas of functionality 
 +  * **on hold** - This branch has to wait for something, usually another PR or something out of our control (ZF3). 
 +  * **question** - An answer from the VuFind dev team is eagerly awaited.
  
 +When using these tags, including a comment or a description that explains the status is welcome and encouraged (especially with on hold and breaking).
 ===== Next Call ===== ===== Next Call =====
  
 The next call will be Tuesday, April 26, 2016 at 9am Eastern Daylight Time (13:00 GMT). The next call will be Tuesday, April 26, 2016 at 9am Eastern Daylight Time (13:00 GMT).
 ---- struct data ---- ---- struct data ----
 +properties.Page Owner : 
 ---- ----
  
developers_call/minutes20160412.1460398580.txt.gz · Last modified: 2016/04/11 18:16 by demiankatz