Samvera Tech Call 2018-11-07

How to connect: https://psu.zoom.us/j/613720745 (link will launch Zoom client – if you do not have Zoom, expand the instructions below)

Time: 9:00am PDT / Noon EDT

Moderator: Andrew Myers

Notetaker: James Griffin

Attendees:

• Gordon Leacock (University of Michigan)
• Frederick Rodriguez (Yale University)
• James Griffin  (Princeton University Library)
• Chris Colvard (Deactivated) (Indiana University)
• Chris Diaz
• Julie Allinson

Agenda

1. Roll call by timezone per following order - ensure notetaker is present (moderator)

   1. folks outside North and South America

   2. Eastern timezone

   3. Central timezone

   4. Mountain timezone

   5. Pacific timezone

   6. folks who were missed or who dialed in during roll call

   7. Remind everyone to sign in on agenda.
   8. Welcome all newcomers!
2. Agenda (moderator)
   1. Call for new agenda items (moderator)
   2. add agenda item here
3. Moderator & notetaker for next time
   
   1. Moderator: 
   2. Notetaker: 
4. After call, this week's notetaker should create the agenda for the next call:

   1. Open template agenda titled "Samvera Tech Call 2018-xx-xx"

   2. Click on ... in the top right corner, and select copy.
   3. Popup will open for location. It should contain: 
      1. Space: Samvera
      2. Parent page: 2018
   4. Select copy. New page should be created.
   5. Modify the title to remove "copy of", update it with the next date, add moderator, notetaker, and any carry-over agenda info. Click Publish.
5. PR Review (Chris Colvard (Deactivated))
   
   1. Review PRs deemed priority from any Samvera project.
   2. Review Hyrax PRs, sorted by 'Least recently updated' and filter out any that don't need to be considered.
   3. Next Facilitator: YOUR NAME HERE

Notes

No Tech. Call was held, instead the Hyrax PR Review was held immediately

Myers: Call for high priority pull requests requiring review?

Colvard: Notes that there are some updates required in relation to the sass Gem (3.7.0)

bootstrap-sass Gem fails compilation due to this update

Release 3.7.1 for sass is needed in order to address this properly

Hyrax Pull Requests

17 are open

Valkyrie Pull Request

chromedriver 

https://github.com/samvera/hyrax/pull/3335

CLA was not legal to sign

https://github.com/samvera/hyrax/pull/3362

If the last comment was a question, it requires a response

https://github.com/samvera/hyrax/pull/3370 was approved

Any progress from Tom Johnson on getting the CircleCI builds to pass?

Stubbed requests need to be torn down for feature tests, WebMock is disabling the requests between suites, but the tests will pass

Will require merges over CircleCI failures (these merges should have a comment detailing this)

https://github.com/samvera/hyrax/issues/3373 was created to address this

https://github.com/samvera/hyrax/pull/3374

Whether this goes into 3.0 or not?

PR is fine

Plan is to integrate for the 3.0 release

https://github.com/samvera/hyrax/issues/3364

In last week's review, attendees skipped all tagged for the 3.0 release

https://github.com/samvera/hyrax/issues/3368

Also requires a check for the CLA

https://github.com/samvera/hyrax/pull/3380

Was already approved, merged during the call

https://github.com/samvera/hyrax/pull/3383

Not sure why I18n.with_local for English is used

Forces into English

smithjp of UNC

Left a comment on the issue (https://github.com/samvera/hyrax/issues/3321)