Versions Compared

Old Version 2

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.

...

See Trading reviews on Pull Requests for how to get immediate attention to that PR!

Notes

  • Reminder that next week this meeting will shift to 14:00UTC (10:00am ET). For Europeans, this means it will be the same local time as this week.  For North Americans, this moves the meeting (back) to one hour earlier (i.e. the same time it was before daylight saving time began).  We'll stay at 14UTC until daylight saving time ends in the Fall.
  • DSpace & DSpace-CRIS Potential Merger - Nothing to discuss. These groups are not far enough along to ask for community feedback or share any early discussions/ideas.  We're still in the early learning phases of the discussion.
  • DSpace 9.0 Release Schedule dates
    • TOMORROW (Friday) is our merger deadline for Feature PRs.  We have a small amount of flexibility in that it may be possible for a few Feature PRs to be merged early next week (if they are showing good progress by the end of this week).  Any Feature PRs not showing progress by end of this week will be rescheduled for 10.0
    • Next Week (March 31 - April 4) -
      • All developers of Feature PRs should add documentation to our 9.0 Documentation (https://wiki.lyrasis.org/display/DSDOC9x)
      • Once all feature PRs are merged, please help to test the system for any major bugs.  This could be simply testing the Sandbox site: https://sandbox.dspace.org , or testing on your own local machine with the latest "main" branch. (Tim will ping developers on Slack next week, hopefully by Weds, when testing would be helpful)
    • April 7-18 is 9.0 Testathon
  • Higher priority dependencies
    • Refactor menu resolvers: https://github.com/DSpace/dspace-angular/pull/3994. 
      • Overall, positive feedback from Tim.  Just an outstanding accessibility issue.
      • Giuseppe notes that he has some concerns about this PR.  He's worried it's "overengineering" and may be problematic.  He will write his concerns down in a review later today
      • Noted that code reviews were due by the code review deadline (March 14).  So, we need to keep that in mind for any late reviews.  Tim agrees, but wants to understand the concerns better.  Maybe there's a way to solve some of them quickly?
      • Tim also notes this might be considered a "nice to have" from a production perspective.  It's a very useful PR for developers, as it makes it easier to customize menus & less likely to encounter merge conflicts when doing so.  But, it doesn't add any real "new features" for production (unless you are modifying these menus in your site)
      • We need to decide quickly what to do about this PR (by early next week at the latest).
    • Migration to NX (https://github.com/DSpace/dspace-angular/pull/4019).
      • Unfortunately no one has had time to review this (too many other open PRs).  This will need to be delayed until the 10.0 release.  However, if we can start to look at it during the 9.0 release process, then hopefully we could consider merger immediately after  the 9.0 release.
  • Review of all open Feature PRs on the 9.0 Board (https://github.com/orgs/DSpace/projects/31)

Action items