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

Update the slack-client dependency #54

Open
ryanguill opened this issue Oct 8, 2017 · 4 comments
Open

Update the slack-client dependency #54

ryanguill opened this issue Oct 8, 2017 · 4 comments

Comments

@ryanguill
Copy link
Owner

We currently use https://www.npmjs.com/package/slack-client but it has been deprecated and now we should be using https://www.npmjs.com/package/@slack/client

https://slackapi.github.io/node-slack-sdk/changelog

I have no idea how much this is going to break but we're pretty far behind at this point.

@ryanguill
Copy link
Owner Author

OT but it really grinds my gears that they pointed the old package to the new github repo.

@atuttle
Copy link
Collaborator

atuttle commented Oct 8, 2017

@yamishi13
Copy link

I can take a look at this if it's ok

@ryanguill
Copy link
Owner Author

sure thing @yamishi13! I am hopeful that everything that would need to be changed would be confined to main.js and plugins/core.js, but I don't know that for sure. Feel free to ping me if you have any questions or need any help. Thanks!

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

No branches or pull requests

3 participants