+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 <[email protected]> wrote: > 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 >
_______________________________________________ Pulp-dev mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-dev
