Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • (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
    1. REMINDER: This meeting switches times to 14:00UTC on April 4.  For those already in daylight savings time (USA), this will be one hour earlier. For Europe, this will be the same time (locally).
    2. Submission PR is still missing comments for the DynamicForms components, wait for those, or move those to a different PR and merge as-is?
    3. MyDSpace PR outstanding comment on the use of the Constants class, see issuecomment-477629344
  • (15 mins) Planning for next week

...

  1. (warning) (REST Contract) Group and eperson management: https://github.com/DSpace/Rest7Contract/pull/41 (Waiting on updates from Ben Bosman )
  2. (blue star) (REST Contract) Configurable Entities (Part 1 - Architecture & Display): https://github.com/DSpace/Rest7Contract/pull/57 (Mark H. Wood - approved., Paulo Graça  - approved)
  3. (blue star) (REST Contract) Minor changes to sync with the mydspace REST implementation https://github.com/DSpace/Rest7Contract/pull/55 (NEW PRBen Bosman , Tim Donohue )
  4. (warning) (REST) EPerson profile PATCH functionality : https://github.com/DSpace/DSpace/pull/2257 (Andrea Bollini (4Science) , Tim Donohue .  This PR currently has minor merge conflicts.)
  5. (blue star) (REST) Configurable Entities (Part 1 - Architecture & Display):  https://github.com/DSpace/DSpace/pull/2376 (Tom Desair reviewed, Paulo Graça , Andrea Bollini (4Science) )
    1. Sub-PR to add more Integration Tests into PR#2376's branch: https://github.com/DSpace/DSpace/pull/2388 (NEW PR - Tim Donohue)
  6. (blue star) (REST) MyDSpace Endpoint PR: https://github.com/DSpace/DSpace/pull/2391 (NEW PR - Ben Bosman , Tim Donohue , Kevin Van de Velde (Atmire) ) - previous discussion here: https://github.com/4Science/DSpace/pull/65
  7. (tick) (blue star) (Angular) Submission implementation: https://github.com/DSpace/dspace-angular/pull/279  (Art Lowel (Atmire) , Paulo Graça - changes requested, Tim Donohue . All change requests implemented, needs final approval
  8. (warning) (Angular) Transfer to .po message format - Initial PR: https://github.com/DSpace/dspace-angular/pull/366 (Paulo Graça - changes requested, Tim Donohue , Art Lowel (Atmire)  - requested updates. Currently failing build.)
  9. (blue star) (Angular) Configurable Entities (Part 1 - Architecture & Display): https://github.com/DSpace/dspace-angular/pull/372 ( Paulo Graça - approved. Noted performance/loading issues. - Art Lowel (Atmire) will look at performance.)
  10. (Angular) Search performance optimizations: https://github.com/DSpace/dspace-angular/pull/374 (Giuseppe Digilio (4Science) , Tim Donohue  - MyDSpace UI needs this) 
  11. (Angular) Bootstrap security update: https://github.com/DSpace/dspace-angular/pull/375 (Giuseppe Digilio (4Science) approved, Tim Donohue - Could be merged immediately)
  12. (Angular) Preview release theme: https://github.com/DSpace/dspace-angular/pull/376 (NEW PRTim Donohue  - Should this be merged into "master" or a "preview" branch? May need to discuss next week?)
  13. (Angular) Fix pagination on the home page: https://github.com/DSpace/dspace-angular/pull/377 (Paulo Graça, (NEW PRGiuseppe Digilio (4Science), Tim Donohue )
  14. (warning) (Angular Bug) https://github.com/DSpace/dspace-angular/issues/368 (Andrea Bollini (4Science),  Giuseppe Digilio (4Science)Tim DonohueBen Bosman Art Lowel (Atmire) )
  15. (Backend) Performance Testing bug: https://github.com/DSpace/DSpace/pull/2379 (Terrence W Brady , Tim Donohue )
  16. (Backend) One Webapp Backend - Initial PR: https://github.com/DSpace/DSpace/pull/2265 (Ben Bosman , Mark H. Wood , Terrence W Brady - tested and found issues in getting RDF/SWORDv1 working in Docker)
    1. This PR will have a follow-up PR to rename the "dspace-spring-rest" webapp to "dspace-server" and update all URL configurations
    1. Docker changes to merge after this PR is merged: https://github.com/DSpace-Labs/DSpace-Docker-Images/pull/96

...

  1. (REST Contract) Edit Homepage News: https://github.com/DSpace/Rest7Contract/pull/45
    1. Delayed until after Preview release. 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.
  2. (REST) Scripts & Processes endpoint: https://github.com/DSpace/Rest7Contract/pull/17
  3. Concurrency in DSpace 7 (or 8).  What do we want to do when multiple editors are editing the same object?  Needs further analysis regarding implementation details
    1. We've decided (in meeting on March 7, 2019) to use ETags to implement concurrency. REST Contract notes on ETags: https://github.com/DSpace/Rest7Contract#etags--conditional-headers
    2. ETags only update of the two fields match. If someone edits first, your edit would fail and you would get a fail response (422?)
    3. ETags seems to have broader support in other REST APIs.  Recommended also by both Art and Andrea.

...