Page tree
Skip to end of metadata
Go to start of metadata

Date

 from 14:00-14:30 UTC

Location: https://lyrasis.zoom.us/my/dspace (Meeting ID: 502 527 3040).

Beta 3 Sprint : May 18-29

This meeting is a short 30 minute check-in which will only involve reviewing of "PRs Needing Review".  There will be no meeting on Thursday, May 21 as it's a public holiday in Belgium (so the Atmire team will be unavailable).

Agenda

Attendees

7.0 Release Goals

These resources define the prioritization and general schedule we are working towards

Current Work

Legend for status icons

(blue star) = Highest Priority tasks (please prioritize these reviews/tasks over others).

(error) = review done, changes were requested or bugs found.

(tick) = review done, approved.

(warning) = review done, merge conflict or other minor changes requests

1 APPROVAL = pull request only requires a single approval to merge.  This is generally reserved for PRs which are either smaller, obvious, and/or bug fixes with tests to prove they work.  

Claim a Ticket!

If you do not have access in JIRA or GitHub to officially claim the ticket you wish to work on, contact Tim Donohue

PRs Needing Review

  1. (REST Contract) (beta 4) (hoping to start development next week - PLEASE REVIEW THIS WEEK) Stats reports https://github.com/DSpace/Rest7Contract/pull/125 (Tim Donohue , Andrea Bollini (4Science) )
  2. (blue star) (HIGH PRIORITY) (REST) (beta 3) Subresources should obey access restrictions (blocks other work) https://github.com/DSpace/DSpace/pull/2726 (Tim Donohue - TEST & APPROVEAndrea Bollini (4Science)- REVIEW BY FRICraig Rosenbeck - (tick))
  3. (NEW) (REST) (beta3) New user registration & forgot password https://github.com/DSpace/DSpace/pull/2763 (Tim DonohueAndrea Bollini (4Science))
  4. (REST) (beta3) Scripts & processes: importing and exporting csv's https://github.com/DSpace/DSpace/pull/2648 (Andrea Bollini (4Science) - REREVIEWTim Donohue - (warning) prior feedback has not yet been addressed )
  5. (REST) Configurable whitelist for "Access-Control-Allow-Origin" header: https://github.com/DSpace/DSpace/pull/2735 1 APPROVAL ((tick) Ben Bosman , Giuseppe Digilio (4Science) - REVIEW)
  6. (NEW) (REST) (beta3) DS-4042: Fix implementation and add IT for wrong PATCH request in submission: https://github.com/DSpace/DSpace/pull/2760 (Tim Donohue - REREVIEWBen Bosman)
  7. (blue star) (NEW) (HIGH PRIORITY) (REST) (beta3) DS-4499: Fix cleanup of test environment in earlier ITs (and performance increase in ITs): https://github.com/DSpace/DSpace/pull/2759 ((tick)Tim Donohue , Ben Bosman - (warning) feedback added)
  8. (NEW) (REST) (beta 3) CC License support: https://github.com/DSpace/DSpace/pull/2764 (Tim DonohueCraig Rosenbeck)
  9. (NEW) (REST) (beta 4) Configuration property retrieval Rest endpoint/Google Analytics support: https://github.com/DSpace/DSpace/pull/27621 APPROVAL (Tim Donohue, POSSIBLY A SECOND REVIEWER)
  10. (Angular) (beta 3) Edit resource policies https://github.com/DSpace/dspace-angular/pull/645 (WAITING ON UPDATE FROM Giuseppe Digilio (4Science) )(Tim DonohueArt Lowel (Atmire) -  (warning) Feedback added,  Julian Timal (eScire) )
  11. (Angular) Login as EPerson  https://github.com/DSpace/dspace-angular/issues/653 ((tick) Tim Donohue , Giuseppe Digilio (4Science) (warning) feedback added,Julian Timal (eScire))
  12. (Angular) (beta3) Scripts & Processes Admin UI https://github.com/DSpace/dspace-angular/pull/636 (Tim DonohueGiuseppe Digilio (4Science)Craig Rosenbeck)
    1. Depends on REST PR #2648 (see above) 
  13. (NEW) (Angular) Fix delay in breadcrumb rendering https://github.com/DSpace/dspace-angular/pull/675 1 APPROVAL (Giuseppe Digilio (4Science)Tim Donohue)
  14. (NEW) (Angular) Fix issue with environment UI variables being ignored https://github.com/DSpace/dspace-angular/pull/677 (Paulo GraçaTim Donohue)
  15. (Backend) (low priority) DS-626 : Exchange usage data with IRUS https://github.com/DSpace/DSpace/pull/2664 ((tick)Craig RosenbeckTim Donohue - REREVIEW)
  16. (Backend) (low priority) (tentative 7.2) DS-4440 GDPR - Anonymize Statistics Feature: https://github.com/DSpace/DSpace/pull/2692 (Andrea Bollini (4Science)Ben BosmanTim Donohue)
  17. (Backend) [DS-4149] porting XOAI additional indexer https://github.com/DSpace/DSpace/pull/2756 (Paulo Graça, Andrea Bollini (4Science) )

PRs Needing Updates  / On Hold

  1. (REST Contract) (beta 3) related to the scripts & processes PR above (related to PR 2648 below) https://github.com/DSpace/Rest7Contract/pull/99 ((warning)NEEDS UPDATES FROM Kevin Van de Velde (Atmire) ) (Andrea Bollini (4Science) - (error) REVIEWEDTim Donohue )
  2. (REST Contract) Language support on the backend (possible new cookie for Angular?https://github.com/DSpace/Rest7Contract/pull/122 ((warning) NEEDS UPDATES FROM Andrea Bollini (4Science) ) (Tim DonohueBen Bosman, Art Lowel (Atmire) )
  3. (REST) (WAITING ON DISCUSSION, SEE 2020-05-18 DSpace 7 Transition BTE to Live Import Mtg(tentative 7.1) [DS-4281]: Metadata suggestions in the live import https://github.com/DSpace/DSpace/pull/2712  (Tim Donohue , Andrea Bollini (4Science))
  4. (REST) (WAITING ON DISCUSSION, SEE Controlled Vocabularies and Authority Control in DSpace 7) (beta 3) Controlled vocabularyMykhaylo Boychuk https://github.com/DSpace/DSpace/pull/2743 (REST Contract #120) (Tim Donohue, Kevin Van de Velde (Atmire) )
  5. (Angular) (WAITING ON PROPOSAL FROM Art Lowel (Atmire) FOR RESOLVING CACHING MISBEHAVIOR IN SUBMISSION UI) Alternative links https://github.com/DSpace/dspace-angular/pull/652 (Giuseppe Digilio (4Science) (warning) feedback added, Tim Donohue - (warning) feedback added)

PRs Coming Soon

  1. (beta 3) REST Language Support on the backend Mykhaylo Boychuk (blocked internally, it is under review by Andrea Bollini (4Science))
  2. (input from Andrea Bollini (4Science) coming?) Support Community or Collection Admins adding/managing Community/Collection Groups. See  DS-4486 - Getting issue details... STATUS  and this early brainstorm REST Contract PR

PRs Merged this week!

  1. (tick) (Angular) (beta 3) Administer Workflow https://github.com/DSpace/dspace-angular/pull/650 
  2. (tick) (Angular) Ensuring environment.ts is present before sync-i18n-files invocation https://github.com/DSpace/dspace-angular/pull/676

Blocked

  1. (Blocked PRs go here)

Delayed / Needs Discussion

  1. Initial Performance Testing from Chris.  Needs revisiting / retesting prior to 7.0. 
    1. https://cwilper.github.io/dspace-perftest/
    2. These performance tests were run prior to the work on "projections" (to limit the data returned by the REST API).  Therefore, it is likely performance is much improved, but needs verification testing.
  2. (REST Contract) Edit Homepage News: https://github.com/DSpace/Rest7Contract/pull/45
    1. Delayed. General agreement (in meeting on March 21, 2019) that storing HTML in metadata fields is not really ideal behavior.  Metadata (from a librarian standpoint) tends to be free of format-related markup (as that allows for easier sharing, understanding of metadata.  Currently Community & Collection homepage information is HTML-based and is stored in metadata that is appropriate for a minor subset of information (like the title) but it is better to move large/rich text to bitstreams.  
    2. Proposal here is to consider storing HTML-based markup (for Site, Community & Collection homepages) in Bitstream(s) associated with the object in question.  May allow for more CMS-lite behavior in the future
    3. Timeline for this is uncertain.  Possibly in 7 or 8. May depend on how/whether it can be scoped.

Notes