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 (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
- 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
- Issue #2760 - Viewers and depositors cannot view show page of admin sets (see issue for proposed fix)
- 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.