+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
