Skip to content

[v13] Port and refactor Mattermost from teleport-plugins (#28989)#29549

Merged
kimlisa merged 3 commits intobranch/v13from
lisa/v13/mattermost
Jul 26, 2023
Merged

[v13] Port and refactor Mattermost from teleport-plugins (#28989)#29549
kimlisa merged 3 commits intobranch/v13from
lisa/v13/mattermost

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Jul 25, 2023

backport of #28989 to branch/v13

manual backport b/c of conflict in pb.go (make grpc), go.sum (go mod tidy) and types file (weird git diff)

@public-teleport-github-review-bot
Copy link
Copy Markdown

@kimlisa - this PR will require admin approval to merge due to its size. Consider breaking it up into a series smaller changes.

* Copy files from teleport/plugins/access/mattermost
* Add github.com/mailgun/holster/v3
* Define mattermost types
* Move StatusFromStatusCode to commons dir
* Make changes/ refactor based on other plugins
  - emit status
@kimlisa kimlisa force-pushed the lisa/v13/mattermost branch 8 times, most recently from aa7c1b3 to 6cfb0f9 Compare July 26, 2023 03:51
Test similarly to slack since they both use commons/app.go
@kimlisa kimlisa force-pushed the lisa/v13/mattermost branch from 6cfb0f9 to 9f27511 Compare July 26, 2023 04:00
@kimlisa kimlisa mentioned this pull request Jul 26, 2023
@kimlisa kimlisa enabled auto-merge July 26, 2023 15:36
@kimlisa kimlisa added this pull request to the merge queue Jul 26, 2023
Merged via the queue into branch/v13 with commit 52b505f Jul 26, 2023
@kimlisa kimlisa deleted the lisa/v13/mattermost branch July 26, 2023 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants