PR#65: Get more than one attachment per comment - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

pagure-importer

#65 Get more than one attachment per comment

Merged Nov 12, 2016 by cverna. Opened Nov 5, 2016 by cverna.

cverna/pagure-importer fix_issue_60 into master

Download 65.patch

cverna commented Nov 5, 2016

Make the comment attachment a list to manage more than one
attachment per comment

Fixes issue #60

Signed-off-by: Clement Verna cverna@tutanota.com

rebased

Nov 5, 2016

pingou commented Nov 7, 2016

Why using the file variable directly?

1 new commit added

File variable is not needed

Nov 7, 2016

2 new commits added

File variable is not needed
Make the comment attachment a list to manage more than one

Nov 7, 2016

cverna commented Nov 7, 2016

Updated :) , thanks

1 new commit added

Added link to copr

Nov 7, 2016

3 new commits added

Added link to copr
File variable is not needed
Make the comment attachment a list to manage more than one

Nov 7, 2016

rebased

Nov 8, 2016

bowlofeggs commented Nov 11, 2016

LGTM.

cverna commented Nov 12, 2016

Thanks

Pull-Request has been merged by cverna

Nov 12, 2016

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.