Allow team as team owner/admin

Bug #474130 reported by Daniel Holbach
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LoCo Team Portal
High
Daniel Holbach

Bug Description

A common setup is like this one: ~ubuntu-cl has ~ubuntu-cl-concilio as owner. LoCo Directory does not recursively check team membership when allowing team info changes.

Related branches

Changed in loco-directory:
importance: Undecided → High
milestone: none → 0.1.1
status: New → Triaged
Changed in loco-directory:
milestone: 0.1.1 → 0.1.0
assignee: nobody → Daniel Holbach (dholbach)
Revision history for this message
Daniel Holbach (dholbach) wrote :

I made it so that the team.owner is checked if it's a team. Let's ignore teams being admin for now, going through the list of members to find admins, then check if they're teams, then check if their members are the ones who want to edit or more sophisticated (more levels of recursion) are too slow.

Changed in loco-directory:
status: Triaged → Fix Committed
Revision history for this message
Daniel Holbach (dholbach) wrote :

This need further work. :-/

To better test this, we should fix the ./admin/ pages to let us enter test teams we're admin of.

Changed in loco-directory:
status: Fix Committed → Triaged
Revision history for this message
Daniel Holbach (dholbach) wrote :

Fix available, added testcase for it.

Changed in loco-directory:
status: Triaged → Fix Committed
Changed in loco-directory:
status: Fix Committed → Fix Released
Revision history for this message
Daniel Holbach (dholbach) wrote :

Still does not work. On it.

Changed in loco-directory:
status: Fix Released → Triaged
Changed in loco-directory:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers