Skip to content

Conversation

grigori-gru
Copy link
Contributor

Hello! I use this methods in my matrix-skype project and it would be great if you add it to your project. I have never used typescript before, probably there are some incorrect interafaces or types.

@demurgos
Copy link
Member

demurgos commented Feb 8, 2018

Thanks for the PR.
I am in the process of updating the conversation API (#87) to type-check the responses and simplify it a bit.
It means that your PR will have to be rebased: I'll take care of it this week-end.

Having all the features already working is a huge help. 👍

@demurgos
Copy link
Member

demurgos commented Feb 13, 2018

I am going to rebase and merge this PR first. The other PR changes too much things around conversations and I need more time to ensure that everything works fine and there are no problems migrating.
It will be safer to merge this PR first and publish it.

- Use `messagesUri.threads` instead of passing empty threadId to
  `messagesUri.thread`.
- Check for existence of location header and conversation id.
@demurgos
Copy link
Member

I just finished reviewing your changes: it's pretty good. I just did a small update in createConversation to avoid using an empty thread id and check for the location header.

@demurgos demurgos merged commit f066982 into ocilo:master Feb 14, 2018
demurgos added a commit to demurgos/skype-http that referenced this pull request Feb 14, 2018
Also adds changelog entries for ocilo#86.

- Closes ocilo/skype-http385
@demurgos demurgos mentioned this pull request Feb 14, 2018
demurgos added a commit to demurgos/skype-http that referenced this pull request Feb 14, 2018
Also adds changelog entries for ocilo#86.

- Closes ocilo/skype-http385
demurgos added a commit to demurgos/skype-http that referenced this pull request Feb 14, 2018
Also adds changelog entries for ocilo#86.

- Closes ocilo/skype-http385
demurgos added a commit to demurgos/skype-http that referenced this pull request Feb 14, 2018
Also adds changelog entries for ocilo#86.

- Closes ocilo/skype-http385
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.

2 participants