-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade libraries to better support MM v6/v7 #78
base: master
Are you sure you want to change the base?
Conversation
* 'master' of github.com:availchet/collabora-mattermost: Enable codeql security-and-quality for pull-requests Misc Fixes
…tions in os and io packages.
…ate golangci-lint
@kendy @mrkara @wget @timar @pedropintosilva P.S. It seems that Jenkins is failing on the |
@availchet Sure. Please let me know which Go and nodejs+npm versions you used for this? If I can build it, I'll merge your PR and deploy it on a Mattermost deployment I have for a SMB IT I manage in order to test it at a larger scale :). |
@wget These are the ones I am using: $ node -v
v16.13.1
$ npm -v
8.1.2
$ go version
go version go1.20.1 darwin/amd64 |
I follow this discussion and can confirm that the test version is working like a charm with Mattermost 7.8.0:
|
@availchet Again, thank you for your hard work! |
Hi @AdmerStroh https://drive.google.com/file/d/1Yp5I5S4Ma_P-YcSMDtkcxKGAwa8FdOuo/view?usp=sharing Creating a new file is realized thru the attachment symbol when writing a new message. Permissions are supported. |
Any progress here? Looking forward to it! |
Summary
package.json
andgo.mod
to support MM v6+.Related Issues
Test Environment
Plugin Download Link for testing
Checklist
make check-style
to check for style errors (required for all pull requests).make test
to ensure unit and component tests passed.