mattermost-plugin-github icon indicating copy to clipboard operation
mattermost-plugin-github copied to clipboard
verified publisher icon mattermost

Avoid double fetching post data

Open jfrerich opened this issue 5 years ago • 2 comments

NOTE: This PR must be merged before code will appear in master. If it is merged, feel free to take!

I suggest we have this component receive the post (or even just postMessage if we don't need anything else) so we can avoid a double fetch to the API to get the post.

Relevant lines of code are linked below.

https://github.com/mattermost/mattermost-plugin-github/pull/123/files/cc4ed0749ae13f9c7ea2a71592e666d7cbfd22aa#diff-71ba01315c638d511aa23811dcd1c081R15

Originally posted by @marianunez in https://github.com/mattermost/mattermost-plugin-github/pull/123

jfrerich avatar Mar 11 '20 01:03 jfrerich

@wiggin77 Should we pick this enhancement

Kshitij-Katiyar avatar Jan 09 '25 11:01 Kshitij-Katiyar

Approved by Eric CC: @wiggin77

Kshitij-Katiyar avatar Jan 17 '25 10:01 Kshitij-Katiyar