This makes sense to me. +1. David
On Tue, May 21, 2019 at 2:54 PM Dana Walker <dawal...@redhat.com> wrote: > +1 null > > Dana Walker > > She / Her / Hers > > Software Engineer, Pulp Project > > Red Hat <https://www.redhat.com> > > dawal...@redhat.com > <https://www.redhat.com> > > > > On Tue, May 21, 2019 at 2:47 PM Tatiana Tereshchenko <ttere...@redhat.com> > wrote: > >> +1 and +1 to allow only null and not both. >> >> Tanya >> >> >> >> On Tue, May 21, 2019 at 7:10 PM Brian Bouterse <bbout...@redhat.com> >> wrote: >> >>> >>> >>> On Tue, May 21, 2019 at 12:25 PM Dennis Kliban <dkli...@redhat.com> >>> 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 unset fields. We discovered this usability problem when >>>> investigating issue with the bindings[0]. >>>> >>>> I propose that we switch all fields in pulpcore's REST API to accept >>>> null instead of empty strings and raise a validation error when an empty >>>> string is passed. >>>> >>>> Does this change make sense? Does anyone have objections? >>>> >>> This makes sense to me. +1 to adopting this in core and all plugins. >>> >>> >>>> [0] https://pulp.plan.io/issues/4676 >>>> >>>> >>>> - Dennis >>>> _______________________________________________ >>>> Pulp-dev mailing list >>>> Pulp-dev@redhat.com >>>> https://www.redhat.com/mailman/listinfo/pulp-dev >>>> >>> _______________________________________________ >>> Pulp-dev mailing list >>> Pulp-dev@redhat.com >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >> _______________________________________________ >> Pulp-dev mailing list >> Pulp-dev@redhat.com >> https://www.redhat.com/mailman/listinfo/pulp-dev >> > _______________________________________________ > Pulp-dev mailing list > Pulp-dev@redhat.com > https://www.redhat.com/mailman/listinfo/pulp-dev >
_______________________________________________ Pulp-dev mailing list Pulp-dev@redhat.com https://www.redhat.com/mailman/listinfo/pulp-dev
