search

gabrieljackson / mattermost-plugin-wrangler

Manage Mattermost Messages Masterfully!
Apache License 2.0
49 stars 13 forks source link

Add wranger post prop when wranger makes posts #173

Closed crspeller closed 6 months ago

crspeller commented 6 months ago

It would be nice to know posts that have been moved by wrangler programmatically.

One case is https://github.com/mattermost/mattermost-plugin-ai would like to ignore wrangler moved posts.

crspeller commented 6 months ago

@gabrieljackson In all the other places I see I think wrangler is posting as itself. Those can be identified by the bot name. Unless I am missing some place.

gabrieljackson commented 6 months ago

Yeah, this is the big one used for moving or copying threads. https://github.com/gabrieljackson/mattermost-plugin-wrangler/blob/master/server/message_helpers.go#L232

If we want to include the props on those then that should take care of it.

crspeller commented 6 months ago

@gabrieljackson Maybe I am misunderstanding. I think the PR already covers that one?

gabrieljackson commented 6 months ago

Sorry, you are right. I misread the change originally. This should be good to go. Thanks!