You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is a little confusion about the configurable values in the annotation, basically, now we can understand some of the values as defaults configurable with the annotation because most of them can be overridden by the configuration override. Should we just keep that ones which we know that does not make sense to override, and others should just be removed from here (and just have them as defaults in the ControllerConfiguraiton interface?
The text was updated successfully, but these errors were encountered:
Was thinking of this, basically having this way set the defaults is not something very unusual. And since we plan to anyways make the annotation optional and the redesign is not straightforward (basically for any of these params, could think of a use case) would rather let this as it is, and close the issue for now.
There is a little confusion about the configurable values in the annotation, basically, now we can understand some of the values as defaults configurable with the annotation because most of them can be overridden by the configuration override. Should we just keep that ones which we know that does not make sense to override, and others should just be removed from here (and just have them as defaults in the
ControllerConfiguraiton
interface?The text was updated successfully, but these errors were encountered: