Comments that contain only spaces are now handled as ScenarioComment.WithoutContent. Refactored, in order to implement validation in a single place.
AutomaticUpdate activity is now only added for scenario, when at least one migration actually applies to that particular scenario (before this fix, the AutomaticUpdate was added for all scenarios when migrations were applied).
Fixed activities of periodic deployment manager:
only finished (with success or with error) data processing should be shown as activity, not deployment errors
added tests
Checklist before merge
[ ] Related issue ID is placed at the beginning of PR title in [brackets] (can be GH issue or Nu Jira issue)
[ ] Code is cleaned from temporary changes and commented out lines
[ ] Parts of the code that are not easy to understand are documented in the code
[ ] Changes are covered by automated tests
[ ] Showcase in dev-application.conf added to demonstrate the feature
[ ] Documentation added or updated
[ ] Added entry in Changelog.md describing the change from the perspective of a public distribution user
[ ] Added MigrationGuide.md entry in the appropriate subcategory if introducing a breaking change
Describe your changes
Fixes related to Scenario Activities:
Comments that contain only spaces are now handled as ScenarioComment.WithoutContent. Refactored, in order to implement validation in a single place.
AutomaticUpdate activity is now only added for scenario, when at least one migration actually applies to that particular scenario (before this fix, the AutomaticUpdate was added for all scenarios when migrations were applied).
Fixed activities of periodic deployment manager:
Checklist before merge