fix: including public channels should check for compliance settings - Githubissues

mattermost / mattermost-plugin-legal-hold

Plugin to create and manage legal holds in Mattermost

Other

5 stars 6 forks source link

fix: including public channels should check for compliance settings #102

Closed fmartingr closed 1 month ago

fmartingr commented 2 months ago

Summary

If the legal hold is created/updated with the exclude public channels disabled (including public channel data) a check is made to ensure that compliance export is enabled
Added validation in the API layer
Removed validation from the KVStore layer (since it's redundant and is not checking what we think it is)
Added a validation for dates (StartDate < EndDate)

Ticket Link

https://mattermost.atlassian.net/browse/MM-59830