#71 Changed generator implementation to list in manage_subscription method
Merged by ralph. Opened by shalini.
shalini/fedora-hubs bugfix/subscriptions  into  develop

Download 71.patch
no initial comment

Looks good to me functionally. Can we break it out onto separate lines a little differently just to for readability? Something like

return [
    '<a href="{link}">{item}</a>'.format(
        link=flask.url_for('hub', name=item),
        item=item,
    ) for index, item in enumerate(items[0:3])
]

yeah, sure, will do it

Looks good. Thanks for the fix @shalini!

Metadata