[StepN|BE] Version Control for Snippets: Remove content presence validation
Now that users will be able to push content to snippet repositories, they can also push empty files to it. Nevertheless, there are two places when this can be conflicting:
- when we create a new snippet we don't allow empty files
- we are double syncing (for now) content pushed through git. This means that, if the user pushes an empty file, when we update the snippet, it will raise a validation error.
We need to remove the content
presence validation. But, in order to have clearer snippets, we should still reject empty files submitted through the UI. For that we can add the HTML attribute required="required"
, have some sort of javascript preventing this, and reject as well the operation in the controller.
Basically we want the same behavior that we have at the moment but without the presence validation. This is a soft requirement. It's not a big deal if the user can somehow create an empty snippet because they will be able to do so through git.
/cc @dsatcher
Edited by Francisco Javier López