Skip to content

Commit

Permalink
Remove "as_user" since it has be deprecated by Slack for new Apps (#61)
Browse files Browse the repository at this point in the history
  • Loading branch information
archive authored Mar 26, 2023
1 parent d9dae40 commit b91c7e2
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ jobs:
slack-channel: C02HEBQP46T
slack-text: "Test 1 - :fire:"
slack-optional-icon_emoji: ":fire:"
slack-optional-as_user: false

- name: Result from "Send Slack Message"
run: echo '${{ steps.send-message.outputs.slack-result }}'
2 changes: 1 addition & 1 deletion .github/workflows/10-slack-fake-build-updates.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on: [push]
jobs:
slack-action:
#if: ${{ false }}
runs-on: ubuntu-20.04
runs-on: ubuntu-lates
name: Test 10 [ubuntu-20.04]

steps:
Expand Down
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,6 @@ jobs:
slack-channel: CPPUV5KU0 #USE CHANNEL ID, NOT CHANNEL NAME, SINCE ID IS USED IN NEW SLACK API's
slack-text: Hello! Something is burning! Or not...
slack-optional-icon_emoji: ":fire:"
slack-optional-as_user: false
- name: Result from "Send Message"
run: echo "The result was ${{ steps.notify.outputs.slack-result }}"
```
Expand Down
6 changes: 3 additions & 3 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ inputs:
slack-blocks:
description: "Blocks"
required: false
slack-optional-as_user:
description: "https://api.slack.com/methods/chat.postMessage#arg_as_user"
required: false
#slack-optional-as_user: - THIS HAS BEEN DEPRECATED BY SLACK - https://twitter.com/SlackAPI/status/1288171767887024130
# description: "https://api.slack.com/methods/chat.postMessage#arg_as_user"
# required: false
slack-optional-attachments:
description: "https://api.slack.com/methods/chat.postMessage#arg_attachments"
required: false
Expand Down
2 changes: 0 additions & 2 deletions integration-test/end-to-end.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ const buildUpdateMessage = require("../src/update-message/build-update-message")

const testSendMessage = async (channel, token, text, optional = {}) => {
const message = buildMessage(channel, text, null, {
as_user: false,
icon_emoji: ":fire:",
...optional,
});
Expand All @@ -23,7 +22,6 @@ const testSendMessage = async (channel, token, text, optional = {}) => {

const testSendReaction = async (channel, token) => {
const message = buildMessage(channel, "Test 2 - testSendReaction 🤓", null, {
as_user: false,
icon_emoji: ":fire:",
});
const messageToReactTo = await apiPostMessage(token, message);
Expand Down

0 comments on commit b91c7e2

Please sign in to comment.