Page History

...

1.  (Angular) Submission implementation: https://github.com/DSpace/dspace-angular/pull/279  (Giuseppe Digilio (4Science)) (Updates requested & fails manual testing)
2. (Angular) Administrative Item Edit (Art Lowel (Atmire) ) (work in progress) (Timeline: Should be ready by Feb 21)
3. (Angular) Browse by date pages (Art Lowel (Atmire) ) (work in progress)  (Timeline: Should be ready by Feb 21)
4. (Angular) Context sensitive admin menus (Art Lowel (Atmire) ) (work in progress)  (Timeline: Should be ready by Feb 28)
5. (Angular) Adding Accessibility via Travis CI  https://github.com/DSpace/dspace-angular/pull/356 (work in progress) (Lower priority)
6.  (REST Contract) Edit Homepage news: https://github.com/DSpace/Rest7Contract/pull/45 (Ben Bosman) (Updates requested from Tim & Andrea) (Lower priority) (Timeline: Should be ready by Feb 28)
7. (REST) Workflow Endpoint: https://github.com/DSpace/DSpace/pull/2312 (Andrea Bollini (4Science) - EARLY Reviews Needed. Reviewers:  Tim Donohue , Ben Bosman Hibernate issues with integration tests.) (Timeline: Should be ready by Feb 21)
8. (REST) MyDSpace Endpoint (Andrea Bollini (4Science) - EARLY Reviews Needed. Reviewers: Ben Bosman , Tim Donohue  (IT missings) on the 4Science repo against the 2312 PR https://github.com/4Science/DSpace/pull/65 (Timeline: Should be ready by Feb 28, maybe before exact date t.b.c on 18th 28 )
9.  (REST) Updating Owning Collections: https://github.com/DSpace/DSpace/pull/2283 (Kevin Van de Velde (Atmire)  - changes requested. Implementation doesn't align with contract) (Timeline: Should be ready a week after the entities are merged)
10.  (REST) Item Mapper: https://github.com/DSpace/DSpace/pull/2282 (Kevin Van de Velde (Atmire)  - changes requested. Implementation doesn't align with contract) (Timeline: Should be ready a week after the entities are merged)
    
11. (Backend) One Webapp Backend - Initial PR: https://github.com/DSpace/DSpace/pull/2265 (Tim Donohue ) (work in progress)

...

• Discussed code comments in tests.
  • Decided that code comments are just highly recommended in tests. They are not required, as sometimes tests are self explanatory (and in Angular code, all tests must "describe" themselves in human terms anyways).  When adding comments, they need not be JavaDocs (or TypeDocs), even inline comments are fine.  Be aware that code reviewers can request code comments for tests. So, if you want a quick review, you may want to add comments.
  • Also noted that PRs that involve bug fixes should provide tests to prove the bug is fixed
  • UPDATE: Tim updated the "PR Creation / Code Review Process" section of our Processes document with these notes: https://docs.google.com/document/d/12BZVIGU9FhaL6Dkotsc603Fh1611mITfARFRWrwTXpo/edit#
• Discussed Release timelines
  • Should we do a Preview release?  That's more of a question for Steering.
  • Major outstanding features:
    
    • Submission and Workflow - Highest priority to complete.  Most of the code already exists in PR form (but has integration tests that need to still be created), so it can get early reviews (see above).
      • 4Science feels all the PRs (for both REST and Angular) should be completed by end of Feb.
      • Code reviews / testing may still take a few weeks, so they should be merged by early to mid March.
    • Entities - Second highest priority. Most of the code already on the "configurable_entities" branches (this branch exists in REST Contract repo, REST repo and Angular repo).  It is immediately reviewable there.
      • All this code has received reviews from the DSpace 7 Entities Working Group (2018-19).  However, we want to give the DSpace 7 team and general Committers time to review it as well, since there are major changes here.
      • Tim will let Committers know this code could use immediate attention.  However, members of the DSpace 7 team should prioritize Submission & Workflow over reviewing Entities – we will ensure this team has time to review Entities as well.
        • We should also make sure to also separate out "suggestions for improvement" or bugs (which can be logged and fixed later) versus actual "blockers" (which need to be fixed more immediately).  We hope that there's more of the former than the latter.
      • Timeline is less nailed down.  Work on the "configurable_entities" branch is not yet complete (but close).  PRs to master do not yet exist.  Once PRs exist, we'll want to allow for a minimum of two weeks for review (exact timeline TBD though).
• Other discussion items were moved to next week.  
  • Discussion on PR#355 (around usability issues) should move to the PR itself.  Art recommends we merge the PR and consider logging the usability issues as a known bug.  The way in which the (upcoming) Admin Edit functionality works also could provide a better option for the future.  We may want to consider a future PR to make this work more like the (upcoming) Admin Edit functionality.  Art will provide more info once Admin Edit PR exists (likely before next meeting).
• Assigned PRs for review (see above).
• Next meeting's notes page already created at 2019-02-21 DSpace 7 Working Group Meeting