This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/44/

Does this work with ilgiz's update_sync code? Is anything other than the display code relying on having the URL? ie, what else could break by suddenly changing the update url during sync?

- Tim Flink


On August 12th, 2013, 12:57 p.m. UTC, Martin Krizek wrote:

Review request for blockerbugs.
By Martin Krizek.

Updated Aug. 12, 2013, 12:57 p.m.

Bugs: 386
Repository: blockerbugs

Description

commit be73d7514172ad122ec969b74a1bac37f14f28f4
Author: Martin Krizek <mkrizek@redhat.com>
Date:   Mon Aug 12 14:52:39 2013 +0200

    Update URLs should comtain the update ID when possible
    
    Fixes: #386

Testing

Run sync without the fix to fetch urls with title in them. Then I run sync with the fix, the sync process updated the urls to contain updateid instead of title.

Diffs

  • blockerbugs/util/update_sync.py (d664839ec1c5979dce980e7baad58154f4622e11)

View Diff