This makes sense to me. +1. David
On Tue, May 21, 2019 at 2:54 PM Dana Walker <[email protected]> wrote: > +1 null > > Dana Walker > > She / Her / Hers > > Software Engineer, Pulp Project > > Red Hat <https://www.redhat.com> > > [email protected] > <https://www.redhat.com> > > > > On Tue, May 21, 2019 at 2:47 PM Tatiana Tereshchenko <[email protected]> > wrote: > >> +1 and +1 to allow only null and not both. >> >> Tanya >> >> >> >> On Tue, May 21, 2019 at 7:10 PM Brian Bouterse <[email protected]> >> wrote: >> >>> >>> >>> On Tue, May 21, 2019 at 12:25 PM Dennis Kliban <[email protected]> >>> 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 >>>> [email protected] >>>> https://www.redhat.com/mailman/listinfo/pulp-dev >>>> >>> _______________________________________________ >>> Pulp-dev mailing list >>> [email protected] >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >> _______________________________________________ >> Pulp-dev mailing list >> [email protected] >> https://www.redhat.com/mailman/listinfo/pulp-dev >> > _______________________________________________ > Pulp-dev mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-dev >
_______________________________________________ Pulp-dev mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-dev
