Ticket #9259 (closed PLIP: fixed)

Opened 7 years ago

Last modified 6 years ago

Group dashboards

Reported by: optilude Owned by: optilude
Priority: major Milestone: 4.0
Component: General Version:
Keywords: Cc: plip-advisories@…

Description

Presuming the portlets infrastructure and dashboard will stay the same in Plone 4, I'd like to propose that we add support per-group dashboards. These would allow the administrator to assign portlets to dashboards on a per-group basis. Users would see on their dashboards:

  • All their personal portlets
  • All portlets for each group they belong to

It should be possible to implement this so that users' ability to manage personal portlets can be turned off completely, whilst still giving them access to their group portlets on the dashboard.

Assigning portlets to the "Authenticated Users" pseudo-group should also work.

The ability for group members to manage their own group dashboards is out of scope, as is the ability for users to move/re-order their own portlets in relation to group portlets.

There is already an implementation of this in collective.groupdashboard. We should probably merge this into plone.app.portlets, since it overrides a few of plone.app.portlets' templates.

Change History

comment:1 Changed 7 years ago by optilude

  • Owner set to optilude

comment:2 Changed 7 years ago by alecm

+1

This seems quite useful, risk-free, and already implemented/tested. Next time write a better PLIP though ;-)

comment:3 Changed 7 years ago by pupq

Very few students seem interested in the dashboards-by-users (it seems that most know that their users won't do this). Having the possibility of by-group may make this feature much more useful to people.

comment:4 Changed 7 years ago by elvix

Dashboard by group is more useful than dashboard by user in intranet settings. We need more portlet variation for these to be very useful though… +1

comment:5 Changed 7 years ago by erikrose

  • Owner optilude deleted

Clearing Owner field of 4.0 PLIPs so we can use it to mean "implementor". (Many of these owners were automatically assigned from choosing a Component that had a default owner.)

comment:6 Changed 7 years ago by smcmahon

  • Cc plip-advisories@… added

comment:7 Changed 7 years ago by optilude

  • Status changed from new to assigned
  • Owner set to optilude

I'll do this work if accepted.

comment:8 Changed 7 years ago by davisagli

My FWT vote: +1.

comment:9 Changed 7 years ago by MatthewWilkes

FWT Vote: +1

comment:10 Changed 7 years ago by rossp

My FWT is +1. I do wish, however, that this was written as a proper PLIP. I'm voting on this pretty much by pedigree and I hate to do that.

comment:11 Changed 7 years ago by raphael

FWT vote: +1

comment:12 Changed 7 years ago by calvinhp

FWT Vote: +1 but ditto on rossp, we have a PLIP format for a reason.

comment:13 Changed 7 years ago by esteele

Approved by FWT vote.

comment:14 Changed 7 years ago by optilude

(In [28571]) Branch for PLIP refs #9259

comment:15 Changed 7 years ago by optilude

(In [28574]) Branch for PLIP refs #9259.

comment:16 Changed 7 years ago by optilude

(In [28575]) Implement PLIP refs #9259

comment:17 Changed 7 years ago by optilude

(In [28576]) Implement PLIP refs #9259

comment:18 Changed 7 years ago by optilude

(In [28577]) Add build + notes for PLIP refs #9259

comment:19 Changed 7 years ago by optilude

This is now ready for review.

comment:20 Changed 7 years ago by ldr

(In [29520]) Review for group dashboard plip. refs #9259

comment:21 Changed 7 years ago by cah190

(In [29625]) Review for group dashboard PLIP. refs #9259

comment:22 Changed 7 years ago by esteele

Your PLIP has been reviewed by the Framework team. Feel free to discuss any suggested changes either here in the PLIP ticket or on the mailing lists. Final deadline for this PLIP is set for September 23.

comment:23 Changed 7 years ago by optilude

(In [29660]) Hide "Dashboard" tab and editable border when there is no "Edit" tab. Refs #9259

comment:24 Changed 7 years ago by optilude

(In [29661]) Remove the 'Manage group dashboards' permission: the existing permission will suffice. Change the UI as suggested by reviews. Refs #9259

comment:25 Changed 7 years ago by optilude

cah190 suggested we make the "Manage group dashboards" permission allow delegation. I fear that's a bigger change than I can take on right now, mainly because it would require a fair amount of UI work. I removed the permission for now, since "Manage portlets" will indeed suffice.

Otherwise, all suggestions have been actioned and this PLIP is ready for merging. There are two post-merge tasks in the 'Plone' egg (because I didn't want to branch that):

comment:26 Changed 6 years ago by cah190

(In [30129]) Updated review for group dashboard PLIP. refs #9259

comment:27 Changed 6 years ago by rossp

FWT vote: +1 for merge

comment:28 Changed 6 years ago by esteele

This PLIP has been accepted for merging into Plone 4.0

The final vote was: Alec Mitchell +1 David Glick +1 Erik Rose +1 Laurence Rowe +1 Matthew Wilkes - Ross Patterson +1

Please merge your branches into the Plone 4.0 head by end-of-day Friday Oct 16. If you need assistance with merging, please contact me.

We'll be assigning a documentation ticket to this PLIP shortly. Please assist the docs team in documenting the changes and new features that this PLIP introduces.

comment:29 Changed 6 years ago by esteele

Please assist the doc team in creating/updating documentation relating to this PLIP. See #9606.

comment:30 Changed 6 years ago by esteele

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [30475]) Removing PLIP branches. Closes #9259

comment:31 Changed 4 years ago by davisagli

  • Component changed from Infrastructure to General
Note: See TracTickets for help on using tickets.