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

Remote bug watches

Bug watches keep track of this bug in other bug trackers.