Skip to content
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

[fix] CI notify-community-server #449

Merged
merged 1 commit into from
Jul 24, 2024
Merged

[fix] CI notify-community-server #449

merged 1 commit into from
Jul 24, 2024

Conversation

phoinixgrr
Copy link
Contributor

@phoinixgrr phoinixgrr commented Jul 24, 2024

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Jul 24, 2024
Alligning the action with the breaking changes introduced under mattermost/action-mattermost-notify#13

Ticket: https://mattermost.atlassian.net/browse/CLD-8100
Signed-off-by: Akis Maziotis <akis.maziotis@mattermost.com>
@phoinixgrr phoinixgrr merged commit 89bcd76 into master Jul 24, 2024
4 checks passed
@phoinixgrr phoinixgrr deleted the CLD-8100-fix-CI branch July 24, 2024 11:51
@phoinixgrr
Copy link
Contributor Author

@hanzei Feel free to chery-pick this to all affected branches. production

@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Jul 24, 2024
@hanzei hanzei restored the CLD-8100-fix-CI branch July 24, 2024 11:58
@hanzei hanzei deleted the CLD-8100-fix-CI branch July 24, 2024 11:58
@hanzei
Copy link
Collaborator

hanzei commented Aug 13, 2024

Hm, this doesn't seem to have done the trick. https://github.com/mattermost/mattermost-marketplace/actions/runs/10353510589 reports an error: You need to provide TEXT or PAYLOAD input

@phoinixgrr Can you please take a look?

@phoinixgrr
Copy link
Contributor Author

Hm, this doesn't seem to have done the trick. https://github.com/mattermost/mattermost-marketplace/actions/runs/10353510589 reports an error: You need to provide TEXT or PAYLOAD input

@phoinixgrr Can you please take a look?

@hanzei
Ah yes. It needs a payload. What to actually send to the channels
https://github.com/mattermost/action-mattermost-notify?tab=readme-ov-file#inputs

What would be that?

@phoinixgrr
Copy link
Contributor Author

Hmm I see.

There is a bash script that needs to be alligned. https://github.com/mattermost/mattermost-marketplace/blob/master/.github/workflows/notify-mm-blog.sh

@phoinixgrr phoinixgrr mentioned this pull request Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants