PR#108: don't accept empty values in custom field values for trac tickets - - Pagure.io

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

pagure-importer

#108 don't accept empty values in custom field values for trac tickets

Merged Jan 30, 2017 by cverna. Opened Jan 21, 2017 by vivekanand1101.

empty_custom_value into master

Download 108.patch

vivekanand1101 commented Jan 21, 2017

Fixes #107

pingou commented Jan 21, 2017

Maybe the .strip() could be done here?

vivekanand1101 commented Jan 21, 2017

It's 81 char already :p

rebased

Jan 22, 2017

puiterwijk commented Jan 23, 2017

For reference, there's also https://pagure.io/pagure/pull-request/1797 that will fix it server-side.

cverna commented Jan 29, 2017

Do we still need this ?

pingou commented Jan 29, 2017

I'd still do it, better safe than sorry :)

cverna commented Jan 30, 2017

so let's merge it :open_hands:

Pull-Request has been merged by cverna

Jan 30, 2017

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.