-
Notifications
You must be signed in to change notification settings - Fork 154
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
values does not work on Slack App approach #171
Comments
Hi @DSdatsme 👋🏻 Thanks for reaching out! I don't see an immediately problem after looking over your code sample and I see you've already tried I'm going to setup an example to recreate your problem and I'll get back to you shortly! Thanks for your patience 🙇🏻 |
Hi @DSdatsme 👋🏻 🌙 I've setup an example repository that uses the Slack GitHub Action in 2 ways: I can confirm that I was able to re-create your error with JSON Payload example. I solved the problem by removing the Example: .github/workflows/slack-message-json.yml#L34-L36:
I believe the examples are slightly out-of-date. They should not use the I'd recommend that you look at the examples in the README.md for Technique 2. They appear to be accurate and should unblock you. Let me know how it goes! |
Heya @mwbrooks , My goal for using |
Or maybe I'll have to wait for Will have to wait until either values or env context feature is released, and any other better workaround you could suggest. |
Hey @DSdatsme, thanks for the response and background on why you wanted to use Please checkout the Contexts documentation because GitHub Workflow inputs are also available: Let me know if that helps! It would be ideal if you can use this repo instead of maintaining your own fork 😄 |
Hey @mwbrooks , I've created a PR to your example repo to explain this with an example. This is the payload template that I used {
"type": "section",
"text": {
"type": "mrkdwn",
"text": "URL: https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }} \n GH actor: ${{ github.actor }},,,,,, GH Input: ${{ inputs.branch }}"
}
} This is what I got as a slack message: Unsure if you want to track it as a separate 2 issues or continue here.
|
Hey @zimeg , yes it's missing some context variables. As I mentioned above, as a workaround, I am using this patched version for myself. |
👋 Hello again! Refreshing myself on this issue, I noticed that #159 was moved to #200 and was included in the It's super neat to see you've been supporting this fix in the meantime! ❤️ I'll close this issue now that support is available in both places, and also wanted to share that Please feel free to follow up with additional ideas on inputs or other questions! And to share a few more resources about expected inputs...
This is unfortunately expected when using
I'm interested in hearing more about this! Please feel free to open a new issue if you're still finding the current features aren't supporting your workflows. I'm not so familiar with |
Thanks for the update @zimeg . For now, I think the Just for a reference on For example: with:
values:
my_button: "URL: https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}" Instead of building URL string templates in JSON, we could just pass the This method would enable users to pass on dynamic variables like md5 hash, of a value instead of storing them in github contexts as env, or any other context type(which becomes available at the job level, that some folks may want to avoid). |
Description
The GitHub actions are giving an error for the
values
field when being used. I've copied the sample code from herehttps://github.com/slackapi/slack-github-action/blob/main/example-workflows/Technique_2_Slack_App/JSON_payload.yml#L45-L47
I am using the OAuth approach (approach 2) of the slack bot to send notifications to slack.
Need to understand if I am missing something or if it's for another setup? because documentation does not mention anything about
values
field.What type of issue is this? (place an
x
in one of the[ ]
)Requirements (place an
x
in each of the[ ]
)Bug Report
Reproducible in:
package version:
tested on
node version:
OS version(s): ubuntu-22.04 (github hosted runners)
Steps to reproduce:
Expected result:
values field should work
Actual result:
getting error
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.
The text was updated successfully, but these errors were encountered: