search

DannyDainton / basic-newman-slack-bot

A basic express app that allows you to run Postman collections against different environments with Newman, straight from Slack.
39 stars 14 forks source link

Use Postman API to find collection & environment in Slack command #7

Open wmedali opened 4 years ago

wmedali commented 4 years ago

Update the htmlextra to new version so no template files are needed. Examples with postman echo collection that includes already test scripts update of readme screenshots

DannyDainton commented 4 years ago

I'll bring this down locally and have a look through it in slow time - not sure how much time I'm going to get but I will get to it as soon as I can.

Really appreciate you submitting it.

wmedali commented 4 years ago

Hello @DannyDainton I'd like to drop another pr to send directly the report file to slack channel and react on message depending on test results. something like valid check emoji if tests pass or X emoji if there are any failures

I thought it would be great if you can check this pr before I send another one ^_^

DannyDainton commented 4 years ago

So sorry @wmedali - I completely forgot to look at this. I will take a look today 😢

wmedali commented 4 years ago

Hey @DannyDainton. Any chance to get this pr completed ? 😂

DannyDainton commented 4 years ago

I did but I couldn't get it working correctly with my Slack workspace when using the Postman API - I then ran out of time :(

I have it working now - It looks like some of the copy on the Slack message has been changed, not sure about that will need to look when I have time. The readme needs to be tidied up, there's a mixture of the new gifs you mentioned with the older ones when running the local collections - The copy on the readme would also need to be updated too.

michael-wilkinson commented 4 years ago

Is it possible to nudge this to get the ability to use API Keys brought into the main project? I'm struggling to piece it together by looking at the pull request :( this is a great help and thanks for all the work so far!

DannyDainton commented 4 years ago

There's parts of the readme that don't make sense after this change and although functional, it seems to work, it's going to tricky for folks to understand how it works, without another review of that file.

4nif commented 3 years ago

@DannyDainton @wmedali Is this PR still valid and can we use it for the Slack integration?

DannyDainton commented 3 years ago

There's somethings that @wmedali needs to do before I merge any of it.

I've not really looked at this repo for a long time, I simply moved onto other things to be honest.

MichaelWWD commented 1 year ago

Hello @DannyDainton , was wondering if we can now use this for the slack integration. This is such a great help and thanks for all the work you've put into it