Versions Compared

Old Version 16

changes.mady.by.user Tim Donohue

Saved on

compared with

New Version Current

changes.mady.by.user Tim Donohue

Saved on

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 (NEW PRPaulo Graça, Giuseppe Digilio (4Science), Tim Donohue )
  14. (warning) (Angular Bug) https://github.com/DSpace/dspace-angular/issues/368 (Andrea Bollini (4Science),  Giuseppe Digilio (4Science), Tim Donohue, Ben 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. (tick) (Backend) Upgrading Solr Server for DSpace - Initial PR: https://github.com/DSpace/DSpace/pull/2058
    1. Initial docs started at Installing DSpace (OLD - to be removed)
  2. (tick) (blue star) (REST) Workflow Endpoint: https://github.com/DSpace/DSpace/pull/2312

...

  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.

...

  1. Submission & Workflow UI / backend
  2. Configurable Entities (from DSpace 7 Entities Working Group (2018-19))
  3. Upgrading Solr Server for DSpace (not required for Preview, as there's no user facing features. However, the earlier we feel comfortable merging this effort the better)

Notes

  • Reminder, next week we move meeting to 14UTC (one hour earlier in USA.  Same time for those in Europe, as they start Daylight savings this weekend)

  • Submission PR discussion: https://github.com/DSpace/dspace-angular/pull/279
    • Major outstanding request is just for TypeDocs (comments) on all Dynamic Forms components
    • Most of these DynamicForms Components existed prior to this PR.  We do not expect those to be fixed by this PR
    • Giuseppe Digilio (4Science) recommends to create a new GitHub issue ticket for commenting all the Dynamic Forms components.  Schedule this for after Preview Release and assign it to Giuseppe.
      • Others agree, this sounds reasonable.  Giuseppe will create the ticket & claim it.
  • MyDSpace PR discussion: https://github.com/DSpace/DSpace/pull/2391
    • Andrea Bollini (4Science) wants to ensure we are in agreement on the implementation strategy. Wants to try and get this PR moved along as quickly as possible
    • Strategy is to keep the existing (new) Constants added in the last PR.  Specifically these 4: https://github.com/DSpace/DSpace/blob/master/dspace-api/src/main/java/org/dspace/core/Constants.java#L58-L76
    • As previously noted by Ben, these new Constants are slightly controversial. The design pattern is not great, as anytime we want to index a new type of Object in Solr, it MUST be added to the list of Constants.  This is because we currently index these Constants in Solr to represent the type (so an Item in Solr says it is type "2", instead of saying it's an "item" or similar)
      • Ben & Tim both agree this is not a good design pattern long term.  However, also both agree it's "good enough" for now, and we can re-discuss this after the Preview Release
    • Next Step: This implementation is fine for now. We'll have to determine a better design pattern approach after Preview Release.  It also may be that (time permitting) we'd have to modify the approach in DSpace 8, we'll have to see.
  • EC2 instance for Docker
    • Terry suggests that, since we are releasing Preview Release via Docker, we might also consider having a Preview "demo site" in EC2 using those Docker instances
    • Tim says that sounds reasonable, We'll take this offline though to discuss further and bring back to group as needed
  • Planning for Preview Release
    • Lieven asks if we can set ourselves an internal deadline.  April 11 might be tough, as both DuraSpace Summit and user group in Germany are going on that week.  April 18 maybe?
    • Tim agrees, April 18 seems like a good goal.
    • Also need to understand our availability in late April and May.  How do we get from Preview to Beta. 
    • Decision
      • April 18 is our new Preview Release goal.  This means finalizing reviews & merging all necessary (blue starred) PRs before April 12 or 15.
      • NEXT WEEK: Discussion of availability in late April / all of May.  Do we have any gaps to fill?  Is a Beta possible by OR2019?