URL repeated in merge proposal status change e-mail notification
Bug #1370527 reported by
Matthew Paul Thomas
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
William Grant | ||
Bug Description
When the status of a merge proposal changes, the e-mail notification looks like this:
> The proposal to merge lp:~larsu/ubuntu-system-settings/dont-use-statusicon
> into lp:ubuntu-system-settings has been updated.
>
> Status: Needs review => Approved
>
> For more details, see:
> https:/
> --
> https:/
> Your team Ubuntu Touch System Settings is subscribed to branch lp:ubuntu-system-settings.
The URL of the merge proposal is included twice, with only one line between them. This is redundant.
It might save time to fix bug 592771 at the same time as this bug.
Related branches
lp:~wgrant/launchpad/misc-fixes
- William Grant: Approve (code)
-
Diff: 523 lines (+136/-85)15 files modifiedlib/lp/app/javascript/sorttable/sorttable.js (+2/-2)
lib/lp/bugs/browser/bugtask.py (+35/-41)
lib/lp/bugs/browser/configure.zcml (+2/-14)
lib/lp/bugs/browser/tests/test_bugtask.py (+39/-1)
lib/lp/bugs/model/bug.py (+10/-3)
lib/lp/bugs/templates/bugtasks-index.pt (+0/-7)
lib/lp/bugs/tests/test_bug.py (+26/-1)
lib/lp/code/emailtemplates/branch-merge-proposal-created.txt (+1/-2)
lib/lp/code/emailtemplates/branch-merge-proposal-updated.txt (+0/-1)
lib/lp/code/emailtemplates/review-requested.txt (+1/-2)
lib/lp/code/mail/tests/test_branchmergeproposal.py (+1/-4)
lib/lp/registry/stories/product/xx-product-index.txt (+12/-0)
lib/lp/registry/stories/teammembership/xx-member-renewed-membership.txt (+2/-2)
lib/lp/registry/templates/product-index.pt (+1/-1)
lib/lp/registry/templates/teammembership-self-renewal.pt (+4/-4)
| description: | updated |
| Changed in launchpad: | |
| assignee: | nobody → William Grant (wgrant) |
| status: | New → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| importance: | Undecided → Low |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r17242 <http://