Expose merge proposal ID on the API and provide contextless URL

Bug #563843 reported by Robert Collins
28
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned
pad.lv
Confirmed
Medium
Unassigned

Bug Description

 affects launchpad-code
 done

I'd like to be able to make a log file name based on a merge proposal.
URL's are a poor fit for this because they are a) long and b) contain /
characters which are filesystem delimiters.

-Rob

Paul Hummer (rockstar)
Changed in launchpad-code:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 563843] Re: please supply db id or other unique simple-string for merge proposals

jml noted that the self_link url contains the db id - indeed, I'm using that
as a workaround. However, the doc in the API doc for branch_merge_proposal
does not advertise this as an ongoing feature it would be safe to depend on.

Revision history for this message
Martin Pool (mbp) wrote : Re: please supply db id or other unique simple-string for merge proposals

Also affects pad.lv, which would like to expose short urls for mps.

IDs are already semi-public and stable because the id is in the email address.

Changed in pad.lv:
status: New → Confirmed
importance: Undecided → Medium
Martin Pool (mbp)
Changed in launchpad:
importance: Wishlist → Low
William Grant (wgrant)
summary: - please supply db id or other unique simple-string for merge proposals
+ Expose merge proposal ID on the API and provide contextless URL
tags: added: api code-review
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers