fractal-analytics-platform / fractal-web

Web client for Fractal
https://fractal-analytics-platform.github.io/fractal-web/
BSD 3-Clause "New" or "Revised" License
6 stars 0 forks source link

Extend default handling from undefined-only to null/empty argument objects #540

Closed tcompa closed 3 months ago

tcompa commented 3 months ago

Given https://github.com/fractal-analytics-platform/fractal-web/issues/538, we have case A (arguments object is "empty" and fractal-web fills the defaults) and case B (arguments objects has values, and fractal-web ignore defaults).

Right now, case A actually requires arguments to be "undefined". We should extend it to cases where the object exists but is either null or {}.

This will also make the sandbox behavior (where undefined is sometimes found) more similar to the webclient (where the backend always sends something for the arguments property).