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

Revert rtm.connect => rtm.start PR #190

Merged
merged 4 commits into from Apr 18, 2019

Conversation

Projects
None yet
2 participants
@acconrad
Copy link
Contributor

commented Apr 16, 2019

Revert to get everyone's apps working again. Keep this locked to a minor version bump and then we'll upgrade to use rtm.connect in a major version bump with a deprecation guide.

acconrad added some commits Apr 16, 2019

@@ -4,7 +4,7 @@ defmodule Slack.Mixfile do
def project do
[
app: :slack,
version: "0.18.0",
version: "0.19.0",

This comment has been minimized.

Copy link
@acconrad

acconrad Apr 16, 2019

Author Contributor

@Slotos this version will be what everyone can target to fix their apps

acconrad added some commits Apr 16, 2019

@BlakeWilliams BlakeWilliams merged commit 522c017 into BlakeWilliams:master Apr 18, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@acconrad acconrad deleted the acconrad:acc-revert-connect branch Apr 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.