MP.queued_revid not set when setting queue status in API

Bug #561170 reported by Robert Collins on 2010-04-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Low
Robert Collins

Bug Description

>>> all_mps[0].self_link
u'https://api.staging.launchpad.net/beta/~vila/bzr/conflict-manager/+merge/22274'
>>> all_mps[0].queued_revid
>>> all_mps[0].queuer
<person at https://api.staging.launchpad.net/beta/~lifeless>

I think it makes sense to grab the approved revid by default when queueing.

Related branches

Robert Collins (lifeless) wrote :

Oh, and its marked readonly, so one can't set this by the API.

Tim Penhey (thumper) on 2010-04-12
tags: added: code-review confusing-ui
Changed in launchpad-code:
status: New → Triaged
importance: Undecided → Low
tags: added: merge-queues
Robert Collins (lifeless) wrote :

Note in particular, that the revid parameter to setStatus exists and is API exposed, I'm saying that the default should be to set it to the reviewed_revision_id

Changed in launchpad-code:
status: Triaged → In Progress
Changed in launchpad-code:
assignee: nobody → Robert Collins (lifeless)
milestone: none → 10.05
status: In Progress → Fix Committed
tags: added: qa-needstesting
Tim Penhey (thumper) on 2010-06-01
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui) on 2010-06-02
Changed in launchpad-code:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers