Vishal
Vishal
@srkgupta Validation is already added in the funcs, you have mentioned. * [Update](https://github.com/mattermost/mattermost-server/pull/20900/files#diff-7cfa0efa32e2bec73ffbb7f70cf2c2bc7ab6afef2330356ca1f0d3b0e4625212R172) * [UpdateNotifyProps](https://github.com/mattermost/mattermost-server/pull/20900/files#diff-7cfa0efa32e2bec73ffbb7f70cf2c2bc7ab6afef2330356ca1f0d3b0e4625212R254) @ashishbhate Added tests and logging
/update-branch
@srkgupta @ashishbhate It's up for review.
@furqanmlk At the very least you can test, whether the `Automatic Direct Message Replies` functionality is still working properly or not. And @srkgupta can confirm, if some other testing is...
> My understanding is that posts from plugins and OAuth apps should count towards the post limit. Considering plugins themselves are part of the limit. So should be their posts.
I'll be adding tests in the next commits.
@neallred It looks like both actions will be needed for files. * "Filtering" for files-search - [figma](https://www.figma.com/file/Aob5ffHtfjKzQoebR93ycA/Cloud-Freemium?node-id=125%3A342171) * "Scrubbing" for remaining cases, i.e., when files are shown along with their...
/e2e-test
* I already added tests for Flagged Posts in `store/storetest/post_store.go#2658`. But there was test missing for search, which I've added now in `store/searchtest/post_layer.go#1851`. * As for search and recent, both...