Setting merge proposal status is awkward when doing reviews via the web UI
Bug #320068 reported by
Jonathan Lange
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Triaged
|
Low
|
Unassigned | ||
Bug Description
Launchpad distinguishes between the 'status' of a review and the status of a merge proposal. Still, there is a conceptual link.
Merge proposal statuses are likely to change as the result of a review, so it makes sense to let people do this on the same page.
| Changed in launchpad-bazaar: | |
| importance: | Undecided → High |
| status: | New → Triaged |
Revision history for this message
| Paul Hummer (rockstar) wrote | #1 |
| Changed in launchpad-code: | |
| status: | Triaged → Fix Released |
Revision history for this message
| Jonathan Lange (jml) wrote | #2 |
| Changed in launchpad-code: | |
| status: | Fix Released → Triaged |
| summary: |
- Allow setting merge proposal status when doing reviews + Allow setting merge proposal status when doing reviews on the web UI |
| Changed in launchpad-code: | |
| importance: | High → Low |
Revision history for this message
| Sidnei da Silva (sidnei) wrote Re: Allow setting merge proposal status when doing reviews on the web UI | #3 |
Revision history for this message
| Jonathan Lange (jml) wrote | #4 |
| summary: |
- Allow setting merge proposal status when doing reviews on the web UI + Setting merge proposal status is awkward when doing reviews via the web + UI |
To post a comment you must log in.
This is possible now, using the 'merge' command. For instance, to approve the review and change the status to Approved, do the following in the email:
review approve
merge approved