Chris Coyne
Chris Coyne
Thanks @FiloSottile - I'll have the net tab open on future tests, but so far I can't get it to happen again.
I'm trying to remove mentions of "password", so let me know specifically places you encounter them. I thought I'd cleaned them up, mostly. I am guilty of using the terms...
I'm ok either way. I wouldn't worry about breaking changes since I think no one is using our module yet, and our README has warnings about its instability. But we...
sure, just make sure it's a big version upgrade since it'll break all existing bots.
I'm ok with tests in the client too, I just feel strongly the test suite on the JS side should run end-to-end tests like it has now. For any time...
@songgao mentioned tracking conversationId internally in the library to manage this. That seems good. the one issue is I don't know how the library knows a possible chat's `conversationId` before...
I think I'd even postpone the single process change for now. It's effectively a performance improvement, which I agree with, but there are more missing features we should probably get...
this may be a lot of work. I'm going to make a shorter-term ticket for ephemeral messages to be stripped out when watching for messages right now.