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

[Bug] Legacy WebHook Slack Migration #38

Closed
ordonezgs opened this issue Jul 22, 2020 · 2 comments
Closed

[Bug] Legacy WebHook Slack Migration #38

ordonezgs opened this issue Jul 22, 2020 · 2 comments
Labels
bug Something isn't working documentation Improvements or additions to documentation APIs or similar NoProgress This will not be worked on

Comments

@ordonezgs
Copy link
Collaborator

if(key.toLowerCase() == "slack")
{
$.ajax({
data: 'payload=' + JSON.stringify({
"text": TextToSend,
"username": ScriptName + " " + GM_info.script.version,
"mrkdwn": true,
"channel": serverDB[localStorage.getItem('WMESTSServer')][key]["chanel_" + chanel],
"icon_emoji": actionicon
}),
processData: false,
type: 'POST',
url: serverDB[localStorage.getItem('WMESTSServer')][key][chanel],
error: function(x, y, z)
{
log('Slack error : ' + x + ' ' + y + ' ' + z);
}
});

Not sure if the script is working well in Slack, because of this documentation talking about:
You cannot override the default channel (chosen by the user who installed your app), username, or icon when you're using Incoming Webhooks to post messages. Instead, these values will always inherit from the associated Slack app configuration.

Please check Legacy Docs:
https://api.slack.com/legacy/custom-integrations/incoming-webhooks
and Up-to-Date Slack WebHook Documentation:
https://api.slack.com/messaging/webhooks

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label bug to this issue, with a confidence of 0.95. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@issue-label-bot issue-label-bot bot added the bug Something isn't working label Jul 22, 2020
@tunisiano187
Copy link
Owner

tunisiano187 commented Jul 23, 2020 via email

@ordonezgs ordonezgs added documentation Improvements or additions to documentation APIs or similar NoProgress This will not be worked on labels Jul 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation APIs or similar NoProgress This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants