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

Date

 from 14:00-15:00 UTC

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

Beta 3 Sprint : June 1-12

Agenda

  • (15 mins) Developer Stand Up - Developers give brief updates on their effort (or their team's effort).

    • Update/see "Current Work" section below based on your status. Please feel free to update prior to meeting.
    • Please highlight any new work (needing reviews/testing), any blockers (for you), and any discussion topics you may have.
  • (30 mins) General Discussion Topics
  • (15 mins) Planning for next week

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 3) Authorization for Downloads of restricted Bitstreams https://github.com/DSpace/Rest7Contract/pull/126 (Andrea Bollini (4Science)- (warning) minor question/feedback to address, Tim Donohue - (warning) minor question/feedback to address)
  2. (REST) (beta3) New user registration & forgot password https://github.com/DSpace/DSpace/pull/2763 (Tim Donohue - (error) Needs major code cleanup, especially in ITs, Andrea Bollini (4Science) waiting for a better status before review)
  3. (REST) (beta3) Scripts & processes: importing and exporting csv's https://github.com/DSpace/DSpace/pull/2648 (Andrea Bollini (4Science) - REREVIEWTim Donohue - (error) Needs to address feedback in original REST Contract, other minor feedback added )
  4. (REST) (beta3) DS-4042: Fix implementation and add IT for wrong PATCH request in submission: https://github.com/DSpace/DSpace/pull/2760 (Tim Donohue - (warning) minor feedback, Ben Bosman)
  5. (REST) (beta 3) CC License support: https://github.com/DSpace/DSpace/pull/2764 (Tim Donohue - REREVIEWCraig Rosenbeck)
  6. (REST) (beta 4) Configuration property retrieval Rest endpoint/Google Analytics support: https://github.com/DSpace/DSpace/pull/27621 APPROVAL (Tim Donohue - REREVIEWCraig Rosenbeck)
  7. (REST) (MERGE) Update JQuery and Toastr used in HAL Browser: https://github.com/DSpace/DSpace/pull/2769 1 APPROVAL (Kevin Van de Velde (Atmire), (tick)Julian Timal (eScire) )
  8. (REST) (beta 3) Performance issues with finding Collections a user can submit to  https://github.com/DSpace/DSpace/pull/2771 (Kevin Van de Velde (Atmire)Tim Donohue - (warning) minor feedback)
  9. (REST) (beta 3) (REVIEW & MERGE) DS-4495 Restricted endpoints are sometimes the only HAL link path to public endpoints REST API v7 https://github.com/DSpace/DSpace/pull/2766 (Tim Donohue - (warning) feedback added, Art Lowel (Atmire) (tick) on the condition that Tim's comment is addressed)
  10. (NEW) (REST) (beta 3)DS-4486 Add support/tests for Collection/Community role-based groups for Community/Collection Admins https://github.com/DSpace/DSpace/pull/2774 (Kevin Van de Velde (Atmire) , Tim Donohue )
    1. Kevin's early PR: https://github.com/DSpace/DSpace/pull/2770 (CLOSE, as included now in #2774)
  11. (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) 
  12. (Angular) Add multi-field sorting https://github.com/DSpace/dspace-angular/pull/634 1 APPROVAL  (Giuseppe Digilio (4Science)(warning) feedback added, Tim Donohue - (warning) questions on how to review/test. No specs in PR)
  13. (Angular) Fix issue with polyfill order in safari and firefox https://github.com/DSpace/dspace-angular/pull/689 1 APPROVAL  (Paulo Graça, Tim Donohue )
  14. (Angular) New User Registration https://github.com/DSpace/dspace-angular/pull/687 (Art Lowel (Atmire) , Giuseppe Digilio (4Science)(warning) better to test once REST PR is merged)
  15. (NEW) (Angular) (beta3) Refactor registry service https://github.com/DSpace/dspace-angular/pull/697 (Tim DonohueGiuseppe Digilio (4Science))
  16. (NEW) (Angular) Use external lib for file-size-pipe https://github.com/DSpace/dspace-angular/pull/698 1 APPROVAL(Tim Donohue)
  17. (Backend) (low priority) DS-626 : Exchange usage data with IRUS https://github.com/DSpace/DSpace/pull/2664 ((tick)Craig RosenbeckTim Donohue - REREVIEW)
  18. (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)
  19. (Backend) [DS-4149] porting XOAI additional indexer https://github.com/DSpace/DSpace/pull/2756 (Paulo Graça (warning) feedback added, Andrea Bollini (4Science) )
  20. (NEW) (Backend) [DS-3940] SHERPA v2 API (master port): https://github.com/DSpace/DSpace/pull/2773 (Andrea Bollini (4Science)(warning) feedback added, Kevin Van de Velde (Atmire))

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) (REST Contract) (beta 4) Stats reports https://github.com/DSpace/Rest7Contract/pull/125
  2. (tick) (Angular) Fix delay in breadcrumb rendering https://github.com/DSpace/dspace-angular/pull/675 1 APPROVAL
  3. (tick) (NEW)(Angular) Fix authentication issue with CSR https://github.com/DSpace/dspace-angular/pull/682
  4. (tick) (Angular) Fix issue with environment UI variables being ignored https://github.com/DSpace/dspace-angular/pull/677
  5. (tick) (Angular) (beta 3) Edit resource policies https://github.com/DSpace/dspace-angular/pull/645
  6. (tick) (Angular) UI does not properly logout when clicking logout https://github.com/DSpace/dspace-angular/pull/691 1 APPROVAL
  7. (tick) (NEW) (Angular) Fix minimist vulnerability https://github.com/DSpace/dspace-angular/pull/6921 APPROVAL

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