feat(chat/messages): Implement retrieving thread messages - Githubissues

superfaceai / station

Comlink metadata for self-integrating applications.

https://superface.ai/catalog

MIT License

22 stars 5 forks source link

feat(chat/messages): Implement retrieving thread messages #230

Open martinalbert opened 2 years ago

martinalbert commented 2 years ago

Description

Extends use-case GetMessages in profile chat/messages with new fields:

listThreadMessages: boolean that represents whether to list messages from channel or thread destination.
threadId: string that represents thread id

Motivation and Context

Implement getting thread messages for slack.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I haven't repeated the code. (DRY)
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

martinalbert commented 2 years ago

Discord is able to get thread messages passing threadId as destination but slack has to specify thread id along with it which is not required in profile.

I can create new use-case for it which would remove listThreadMessages but discord use-case would be duplicate of its usecase GetMessages 🤷‍♂️