Skip to content
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

Updates #11

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

Updates #11

wants to merge 7 commits into from

Conversation

crisward
Copy link

fixed #7
Added curl installer, incase user doesn't have it
Added slack logo
Updated readme to show global url option
I've tested this in dokku 0.4.4. works well, don't have access to 0.3.x so not sure if I broke anything for users of older versions.

@matt-oakes
Copy link
Contributor

Hi @crisward

Thanks a lot for the pull request!

I've not yet had a chance to test the changes, however, from reviewing can you make a few changes for me:

  • Remove the Slack logo. While I like it, it's against the Slack brand guidelines to edit their logo in this way. I'm sure it wouldn't matter, but I'd rather not risk it.
  • Can you please squash the commits into a single commit.

Thanks again for the PR. I'll check out the functionality once I have a bit of time. That will likely be either later this week or next week, as I'm away over the weekend.

Matt

@crisward
Copy link
Author

crisward commented May 4, 2016

ok, will do

@israelroldan
Copy link

@crisward sorry to revive an old thread... but are there any plans to merge this PR? if you don't have time I can gladly take over it

@crisward
Copy link
Author

crisward commented Mar 6, 2017

Sorry, I didn't get chance to remove my logo. Just kept on using my fork. https://github.com/crisward/dokku-slack

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

Successfully merging this pull request may close these issues.

Global webhook URL not returned if querying a single app
3 participants