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

Slack Node extended #1238

Closed

Conversation

bf-ascherren
Copy link
Contributor

@bf-ascherren bf-ascherren commented Dec 8, 2020

https://community.n8n.io/t/slack-node-conversations-members-reactions-add/3066

I added:

  • users.info
  • users.getPresence
  • conversations.members
  • reactions.add
  • reactions.remove
  • reactions.get

Increased limit of getChannels from default 100 to 1.000 as my company for example has ~400 public channels.

@CLAassistant
Copy link

CLAassistant commented Dec 8, 2020

CLA assistant check
All committers have signed the CLA.

@bf-ascherren bf-ascherren marked this pull request as ready for review December 8, 2020 10:47
RicardoE105 added a commit that referenced this pull request Dec 8, 2020
@RicardoE105
Copy link
Contributor

@bf-ascherren got reviewed and it seems to be working as expected. Made some small changes though. Thanks for your contribution. We will let you know when is released.

janober added a commit that referenced this pull request Dec 13, 2020
* Added reaction method to message

* Increased limit for channels on getChannels, removed unused fields in the code

* Using own operation for reactions

* Registering reaction fields and operations

* Added Operation "Reaction" to Slack.node.ts

* Fixing variable name for emoji

* now removing reaction on "remove" instead of "add"

* Using GET for reactions.get and passing arguments as query

* Added members operation

* Fixed typo in timestamp

* Added user.info and user.getPresence

* Fixed: wrong operation name

* ⚡ Improvements to #1238

* ⚡ Add field resolve data when retrieving channel members

* ⚡ Minor improvements to Slack-Node

Co-authored-by: Andreas Scherren <ascherren@brightfuture.de>
Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com>
@janober
Copy link
Member

janober commented Dec 13, 2020

Thanks a lot. Got merged with #1239 and will be released with the next version.

@janober janober closed this Dec 13, 2020
@janober
Copy link
Member

janober commented Dec 16, 2020

Got released with n8n@0.98.0

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