toko-bifrost / ms-teams-deploy-card

Microsoft Teams Deploy Card
MIT License
96 stars 77 forks source link

Show cards logic issue #29

Closed uncledru closed 4 years ago

uncledru commented 4 years ago

Setting both show-on-exit and show-on-failure sets the configuration to not show card upon job exit. See images below.

image

image

The issue seems to lie somewhere here

I can contribute if needed :)

aldwyn-acn commented 4 years ago

Hi @uncledru, I've made clear in the README the following descriptions 😁

| show-on-exit | false | true | Show an MS Teams card upon exiting this Github Actions job, regardless if it's a successful or a failed exit | | show-on-failure | false | false | Show an MS Teams card upon exiting this Github Actions job and status is FAILURE; This will override any boolean value of show-on-exit, e.g. with show-on-exit=false and show-on-failure=true, the notification card will still proceed upon job failure |

uncledru commented 4 years ago

Got it. Thanks @aldwyn-acn! If I'm understanding correctly -- I need to set show-on-exit to false and show-on-failure to `true

to get both failure and success notifications on exit?

aldwyn-acn commented 4 years ago

@uncledru to get both failure and success notifications on exit, you only have to set show-on-exit to true. Or don't set any value on it at all because its default is already true. Don't mind setting the show-on-failure.

uncledru commented 4 years ago

Awesome. Thank you! Loving the action :)