Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-22 Thread Dennis Kliban
Thanks everyone for the feedback. I made a PR[0]. [0] https://github.com/pulp/pulpcore/pull/135/ On Wed, May 22, 2019 at 7:04 AM Ina Panova wrote: > +1 to the suggestion. > > > > Regards, > > Ina Panova > Senior Software Engineer| Pulp| Red Hat Inc. > > "Do not go where the path may le

Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-22 Thread Ina Panova
+1 to the suggestion. Regards, Ina Panova Senior Software Engineer| Pulp| Red Hat Inc. "Do not go where the path may lead, go instead where there is no path and leave a trail." On Tue, May 21, 2019 at 9:10 PM David Davis wrote: > This makes sense to me. +1. > > David > > > On Tue,

Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-21 Thread David Davis
This makes sense to me. +1. David On Tue, May 21, 2019 at 2:54 PM Dana Walker wrote: > +1 null > > Dana Walker > > She / Her / Hers > > Software Engineer, Pulp Project > > Red Hat > > dawal...@redhat.com > > > > > On Tue, May 21, 2019 at 2:47 P

Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-21 Thread Dana Walker
+1 null Dana Walker She / Her / Hers Software Engineer, Pulp Project Red Hat dawal...@redhat.com On Tue, May 21, 2019 at 2:47 PM Tatiana Tereshchenko wrote: > +1 and +1 to allow only null and not both. > > Tanya > > > > On Tue, May 21, 201

Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-21 Thread Tatiana Tereshchenko
+1 and +1 to allow only null and not both. Tanya On Tue, May 21, 2019 at 7:10 PM Brian Bouterse wrote: > > > On Tue, May 21, 2019 at 12:25 PM Dennis Kliban wrote: > >> Pulp 3's REST API does not accept Null as a value for a string field. The >> only way for a user to unset a string field is

Re: [Pulp-dev] breaking change: accept Null instead of empty string

2019-05-21 Thread Brian Bouterse
On Tue, May 21, 2019 at 12:25 PM Dennis Kliban wrote: > Pulp 3's REST API does not accept Null as a value for a string field. The > only way for a user to unset a string field is to pass in an empty string. > This makes it awkward to integrate Pulp with other systems that use Null to > represent

[Pulp-dev] breaking change: accept Null instead of empty string

2019-05-21 Thread Dennis Kliban
Pulp 3's REST API does not accept Null as a value for a string field. The only way for a user to unset a string field is to pass in an empty string. This makes it awkward to integrate Pulp with other systems that use Null to represent unset fields. We discovered this usability problem when investig