Update PersonNotification to support teams
Bug #740559 reported by
Curtis Hovey
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Unassigned |
Bug Description
Before bug 736421 can be addressed, there are several infrastructure issues that must be solved first:
* Person.merge needs send success emails to teams.
* PersonNotificat
* Team.getTeamAdm
* The job.getErrorRec
Related branches
lp:~sinzui/launchpad/person-merge-job-1
- Robert Collins (community): Approve
-
Diff: 385 lines (+187/-23)10 files modifiedlib/lp/registry/emailtemplates/team-merged.txt (+8/-0)
lib/lp/registry/interfaces/personnotification.py (+9/-1)
lib/lp/registry/interfaces/persontransferjob.py (+3/-0)
lib/lp/registry/model/person.py (+13/-9)
lib/lp/registry/model/personnotification.py (+21/-6)
lib/lp/registry/model/persontransferjob.py (+8/-1)
lib/lp/registry/scripts/personnotification.py (+3/-2)
lib/lp/registry/tests/test_person_merge_job.py (+18/-4)
lib/lp/registry/tests/test_personnotification.py (+38/-0)
lib/lp/registry/tests/test_team.py (+66/-0)
Changed in launchpad: | |
status: | In Progress → Fix Committed |
tags: | added: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
Changed in launchpad: | |
assignee: | Curtis Hovey (sinzui) → nobody |
To post a comment you must log in.