mattermost / mattermost-plugin-legal-hold

Plugin to create and manage legal holds in Mattermost
Other
5 stars 6 forks source link

Changes in date validation contains some issue #111

Open AayushChaudhary0001 opened 1 month ago

AayushChaudhary0001 commented 1 month ago

@fmartingr While testing the master branch, especially for the changes in PR #102 , I found an issue where the dates which does not exist are also getting saved. After saving them, the end date time is shown as never s the job will run always and will never end. For more info, please look at the below screenshots.

Date which does not even exist:- image

After saving such date:- image

While we have a feature if a user enters and invalid date, then the option to update legal hold is disabled, this can be consistent across such cases as well if we can handle this.