Contribute to the DSpace Development Fund
The newly established DSpace Development Fund supports the development of new features prioritized by DSpace Governance. For a list of planned features see the fund wiki page.
Date
Discussion (combined Angular UI & REST meeting)
Attendees
Notes
- General updates (from Tim)
- Outreach meeting yesterday. Primarily gave updates on our work, and noted that UI screens for Submission & Search will be coming in the next few months.
- In meantime, encouraged them to keep gathering UI examples of inspirational design and functionality, especially in the areas of AuthN/Z
- Next few weeks schedule for Tim
- Out of office Nov 13-17 (at DuraSpace staff meetings)
- USA Thanksgiving holiday Nov 23-24
- Based on the above, we'll skip next week's meeting, and have our next meeting on Tues, Nov 21 at 15:00UTC. We will return to our normal schedule the following week (starting with Thurs, Nov 30th).
- Outreach meeting yesterday. Primarily gave updates on our work, and noted that UI screens for Submission & Search will be coming in the next few months.
- Angular UI Team updates (from Art)
- Merged big PR: https://github.com/DSpace/dspace-angular/pull/180
- After merger, Art noticed every click on pagination triggered a fade-in
- Fix is in https://github.com/DSpace/dspace-angular/pull/198 (but still a WIP, as there still seems to be more frequent "fade-in"s than expected)
- In Progress. This needs updating based on latest master: https://github.com/DSpace/dspace-angular/pull/191
- Lotte's sidebar PR: https://github.com/DSpace/dspace-angular/pull/192
- Art has reviewed. it looks good.
- Everyone is OK with merging this, as it's a blocker for other work. If any bugs are later discovered, we can create new tickets for them
- Upgrading to Angular 5 release: https://github.com/DSpace/dspace-angular/issues/194
- Seems minor, but a few questions.
- Art is working on this in a branch.
- Giuseppe's PR: https://github.com/DSpace/dspace-angular/pull/196
- Art couldn't get this working correctly, but isn't sure he did it right
- Giuseppe responded, and said it looks like he was using the abstract class instead of the actual class.
- Art will retest
- Other feedback in PR itself
- JSON Patch vs JSON Merge Patch:
- Art reviewed. Good overview of both at http://erosb.github.io/post/json-patch-vs-merge-patch/
- Art thinks the JSON Patch format will be easier to implement on Angular side of things.
- Giuseppe notes that Andrea also said he prefers JSON Patch for REST API.
- Sounds like we have a decision. Art will add his feedback to the ticket for Andrea.
- Merged big PR: https://github.com/DSpace/dspace-angular/pull/180
- REST API Updates
- PR was merged https://github.com/DSpace/DSpace/pull/1866
- Tom's new stateless auth PR is looking good. Feedback welcome: https://github.com/DSpace/DSpace/pull/1873
- Currently this is failing tests. Seems to be WIP
- Submission via REST PR is getting close: https://github.com/DSpace/DSpace/pull/1852
- Tom has been giving feedback. Tim will also respond to latest feedback this week.
- Early wiki docs now at Configuration changes in the submission process
- REST contract PR needs reviews: https://github.com/DSpace/Rest7Contract/pull/15
- Next Meeting will be Tues, Nov 21 at 15:00UTC via Google Hangouts
Overview
Content Tools