Skip to content

Add a user attribute to Message #168

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

Merged
merged 1 commit into from
Apr 18, 2018
Merged

Add a user attribute to Message #168

merged 1 commit into from
Apr 18, 2018

Conversation

romainrichard
Copy link
Contributor

Credit goes to #110, but since there has been no activity to fix the PR in more than 6 months I'm submitting a similar PR.

Because of #167, it's hard to tell whether or not my change did anything wrong.
Wasn't sure about writing tests either since there doesn't seem to be tests for get_channel which is similar to the new get_user in this PR, so I didn't write any.

@alairock
Copy link

The maintainer needs to add other maintainers to help him if he doesn't have time for it anymore. This project is an excellent project and really needs love. If he sees this, I am happy to help, just let me know.

I would merge the above in, in a heartbeat. Nice work.

@amuraru
Copy link

amuraru commented Apr 11, 2018

Tried to reachout to the maintainer but got no answer. What I did was to fork the repo, created a new pypi package: slacknotng and pushed there a new version. If anybody is interested in helping maintaining this pls pm me and I can add him a maintainer in github and pypi

@lins05 lins05 merged commit 7e25086 into scrapinghub:develop Apr 18, 2018
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.

4 participants