Andrew Bulat
Andrew Bulat
Some comments before review: Should we add the same for Annotations while we're at it? And callback types seem to mismatch, tests are not happy. Also should discuss if there...
> Something I noticed when testing this branch for Msgpack requests. The SDK currently isn't setting the correct content type header of application/msgpack it was being set to JSON still...
> Can we try running these tests with msgpack to see if it works? I did find this ticket which may indicate an issue with msgpack over rest currently: https://ably.atlassian.net/browse/PUB-1835...
Hello @catpddo ! Sorry for the delay. I looked into this but couldn’t reproduce the `EvalError: Code generation from strings disallowed for this context` error you described, either with Cloudflare...
Hey @ITBoomBKStudio , thank you for reporting this. We can confirm that there is an inconsistency between the TypeScript type definitions and the runtime validation. Both `undefined` and `null` should...
Hi @abmercy035 ! Are you using the ably-js v2? If so, starting with version 2, ably-js no longer offers a choice between a callbacks-based API and a promises-based API, and...