-
Notifications
You must be signed in to change notification settings - Fork 164
Open
Description
This package should really be using promises as most calls are asynchronous.
This wasn't made clear in the documentation, but I realized it when I performed an expensive operation and the connection closed before it could be completed.
For now I need to wrap the calls in promises myself and resolve/reject in the callback like so:
await new Promise((resolve, reject) => {
mixpanel.people.set_once(identifier, data, (e) => {
if (e) {
reject(e);
}
resolve(null);
});
}mogol, nathankleyn, jorgealemangonzalez, Flamenate and mpokryvaJudge-Paul, nathankleyn, jorgealemangonzalez, Flamenate and mpokryva
Metadata
Metadata
Assignees
Labels
No labels