Samvera Tech Call 2018-03-21
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: @justin
Notetaker: @Chris Colvard (Deactivated)
Attendees:
@Jim Coble (Duke)
@Glen Horton (Deactivated) (Cincinnati)
@James Griffin (Princeton University Library)
@justin (Stanford)
@cam156 (PSU)
@Lynette Rayle (Cornell)
@Andrew Myers (WGBH)
@Chris Colvard (Deactivated) (Indiana University)
@Julie Allinson (CoSector, UoLondon)
@jrudder (UNC-CH)
@nesovi (UVA)
@Kevin Musiorski (AIC)
@tamsin woo (DCE)
you!
Agenda
Roll call by timezone per following order - ensure notetaker is present (moderator)
folks outside North and South America
Eastern timezone
Central timezone
Mountain timezone
Pacific timezone
folks who were missed or who dialed in during roll call
Welcome all newcomers!
Agenda (moderator)
Call for new agenda items (moderator)
Hyrax 2.1 release update
Testing
RC2?
HELP NEEDED! Project Board
Bugs requiring feedback
Issue #2760 - Viewers and depositors cannot view show page of admin sets (see issue for proposed fix)
Issue #2796 - Optionally hide private files
Analytics Update (@Nestor Walker)
Status
Need review for merge to master
LDCX and Samvera Dev Congress
add agenda item here
Notetaker and moderator for next time
Notes: @Glen Horton (Deactivated)
Moderate: @cam156
After call, this week's notetaker should create the agenda for the next call.
Notes
Hyrax 2.1 release update
First round of testing done. Need people to help fix bugs. See Project Board for prioritized list.
Bugs requiring feedback
Issue #2760 - Viewers and depositors cannot view show page of admin sets (see issue for proposed fix)
Admin sets didn’t have access controls before but now do because admin sets are now collections
Currently admin sets listed on dashboard but can’t navigate into them due to lack of read privileges
Proposal: grant access to viewers and depositors explicitly added to admin set (more details on ticket: https://github.com/samvera/hyrax/issues/2760#issuecomment-374981473)
Requires migration to back fill permissions on admin sets. This could be easily added to existing migration service.
Ran out of time for discussion. Lynette will write email to samvera-tech on this.
Issue #2796 - Optionally hide private files
Configuration option to toggle between existing behavior of obscured view and hide filesets the user doesn’t have access to
Configurable option seems best we can do right now
Is this really a blocker? Or maybe it is a new feature?
No objections to removing blocker status
Steve agreed on slack
Analytics Update (@Nestor Walker)
Last week of planned sprint
PR needs review for merge to master
Changes include new backend code for storage of statistics and some better reporting (rest will have to be in future sprint)
There was decision to store download counts locally instead of sending them to google analytics or Matomo.
Is this too destabilizing to merge to master given push for 2.1 release?
PR has feature toggle (off by default) to enable new functionality
Should a arelease branch be cut?
Would mean more work porting bug fixes between branches
If a PR has already been merged to release branch then maybe it doesn't need a PR to merge to master
Is there a problem with merging this to master?
Needs DB migration to create new tables
Hyrax 2.1 already requires DB migration so wouldn't be adding any new requirement.
Lynette is nervous about anything that might slow down release.
Two options: Merge into master or make a release branch
Time ran out for discussion.
LDCX and Samvera Dev Congress
Next week
Need to register for Dev Congress by end of today
Agenda is on the wiki so please add your ideas: March 2018 Developer Congress Meeting Agenda
No tech call next week due to LDCX. Next call is 4/4.