Open SteelWagstaff opened 7 months ago
This has an open draft PR.
Currently, the PR includes the new column for each book listed in the network sites page. The current state of the permissions blocks Institutional Managers from accessing the page though (you don't have permission to view this page). We will need to improve our permission checks so it works when Pressbooks Network Analytics is active but also when it's not.
Working:
There are some pending things to work on:
wp_cache_flush_group
call which feels weird)This task was underestimated since I already spent more effort that we anticipated for this task. We decided to move this task to future sprints, so I will consider in our metrics that we spent 5 points, but I would estimate the remaining effort to be another 3 to 5 points.
As an institutional manager on a self-hosted open source PB network (i.e. one without pressbooks-network-analytics), I should see a book list with information specific to my institution, instead of the full network book list.
See https://www.figma.com/file/hAvkPJpDgJDDyh8g358AqR/PB-Site-Adjustments?type=design&node-id=34-10598&mode=design&t=6NpK2f6ILNHepftM-0
This should look like the current network book list, but with a new column called 'Institution'. As an institutional manager, I should only be presented with the books which belong to my institution. If I try to view the full network book list, I should see a 'you don't have permissions to do this' style message.
DoD:
Additional desired changes:
wp-admin/network/site-users.php?id=3
toSITETITLE/wp-admin/users.php
Edit
link beneath the book title in the institutional manager's book list (both in pressbooks-network-analytics and the default WP book list)