link 'list reminders' back to 'view reminders' fixes Issue #100

scottleedavis · May 17, 2019 · 87518d0 · 87518d0
1 parent e725c04
commit 87518d0
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 4 deletions.
diff --git a/plugin.json b/plugin.json
@@ -2,7 +2,7 @@
     "id": "com.github.scottleedavis.mattermost-plugin-remind",
     "name": "Remind Bot Mattermost Plugin",
     "description": "Sets Reminders",
-    "version": "0.2.5",
+    "version": "0.2.6",
     "server": {
         "executables": {
             "linux-amd64": "server/dist/plugin-linux-amd64",

diff --git a/server/Gopkg.lock b/server/Gopkg.lock
diff --git a/server/http.go b/server/http.go
@@ -178,7 +178,8 @@ func (p *Plugin) handleViewEphemeral(w http.ResponseWriter, r *http.Request) {
 		writePostActionIntegrationResponseError(w, &model.PostActionIntegrationResponse{})
 		return
 	}
-	p.ListReminders(user, "")
+	p.API.SendEphemeralPost(user.Id, p.ListReminders(user, request.ChannelId))
+
 	writePostActionIntegrationResponseOk(w, &model.PostActionIntegrationResponse{})
 
 }

diff --git a/server/http_test.go b/server/http_test.go
@@ -104,6 +104,7 @@ func TestHandleViewEphmeral(t *testing.T) {
 		api.On("GetUserByUsername", mock.Anything).Return(user, nil)
 		api.On("KVGet", user.Username).Return(stringReminders, nil)
 		api.On("CreatePost", mock.Anything).Maybe()
+		api.On("SendEphemeralPost", mock.Anything, mock.Anything).Return(nil)
 
 		return api
 	}

diff --git a/server/manifest.go b/server/manifest.go
@@ -5,5 +5,5 @@ var manifest = struct {
 	Version string
 }{
 	Id:      "com.github.scottleedavis.mattermost-plugin-remind",
-	Version: "0.2.5",
+	Version: "0.2.6",
 }