Michael Kochell
Michael Kochell
To solve the issue of 4 requests being done simultaneously, maybe the UI can issue a single request first for this purpose, and then follow up with the 4 requests...
If it's an absolute requirement that we refresh only once per user, we can also use the `cluster.Mutex` function exposed [here](https://github.com/mattermost/mattermost-plugin-api/blob/73008cfda2fbefe305f704eeb45975fabe39d7f2/cluster/mutex.go#L41). For the mutex's key, we can use `userID +...
> There are also downsides to adding the endpoint, as it adds responsibility onto the frontend and the API I think we're good with the mutex on the backend, no...
> Also I see the test is failing. I think it's just something in the test code that needs to be corrected for my changes, looking into it now. It...
@sanjaydemansol Using `title` sounds good
When a plugin is installed, its config structure is not populated at first. As noted in the issue description, the admin needs to submit the plugin's config form from the...
Looks like this may already be fixed. @DHaussermann to check on this
> Changed the messages, anything else to do? @MatthewDorner Make sure you either mention someone when asking a question or re-request review from the person if that's the context of...
I thought about the localization piece some more. Let's remove it from this PR, partially to narrow the scope a bit, but also because it is a larger task to...
@hanzei Maybe we should reconsider how we handle the non-configured state of the plugin. Instead of aborting the startup, we could have the plugin in an "inactive" state while still...