Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wmedali
Copy link

@wmedali wmedali commented Apr 2, 2020

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

wmedali added 3 commits April 2, 2020 10:45
add working server for tunnel command
delete template files
README.md Outdated Show resolved Hide resolved
app.js Outdated Show resolved Hide resolved
add dotenv dependency and load api key from environment file.
Update doc as well
@DannyDainton
Copy link
Owner

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
Copy link
Author

wmedali commented Apr 9, 2020

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
Copy link
Owner

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

@wmedali
Copy link
Author

wmedali commented Jun 18, 2020

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

@DannyDainton
Copy link
Owner

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
Copy link

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
Copy link
Owner

DannyDainton commented Sep 30, 2020

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
Copy link

4nif commented Mar 30, 2021

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

@DannyDainton
Copy link
Owner

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
Copy link

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants