WS-2215 Editor modal scrubbing

The editor modal was allowing empty content to persist to the database. This was because the modal actually saves the new content and if this was an empty object (by leaving the modal open or mischievous behavior) then it would wipe the contents of existing database entry.

The last time we saw this we prevented one of the endpoints from persisting empty data. This MR does that for the capability_version_files table and puts a stop-gap on the UI by not allowing an empty object from being sent to the endpoint.

The behavior is a little odd if someone actually WANTS to clear file content by submitting an empty form, but I don't anticipate that happening (some user will prove me wrong).

assigned to @dnemergu

Just checking, do we want to leave this off?

On dev, I would think so. I don't see much benefit to minimization unless others think so. Could be a good team vote.

oh, it was hiding that this was profile specific

changed this line in version 4 of the diff

approved this merge request

added 1 commit

• e867da4a - Added a note for our future selves

Compare with previous version

added 11 commits

• e867da4a...305a5b40 - 7 commits from branch 2.8.2.3-DEVELOPMENT
• 1e63d35b - Disabling webpack on dev to simplify JS debugging
• 88ab7590 - Preventing the save button from persisting an empty form on the editor modal
• 41c4def6 - Checking that the request body has content before saving a capability version file
• 4029831c - Added a note for our future selves

Compare with previous version

added 1 commit

• 24c83f3c - Revert "Disabling webpack on dev to simplify JS debugging"

Compare with previous version

approved this merge request

merged

mentioned in commit aa0c5374