search

amplitude / Amplitude-Node

Server-side Node.js SDK for Amplitude
MIT License
67 stars 20 forks source link

Node JS SDK minimal version #1

Closed haoliu-amp closed 4 years ago

haoliu-amp commented 4 years ago

Looks like a good start. I'm not entirely sure the inheritance pattern has a payoff here, but it doesn't seem too complicated here yet.

Is there a doc or ticket I missed with the planned API? Particularly curious about how the Identify API will work.

Is it going to send requests to the HTTP-API-V2?

yep, now I am using v2. I think v2 is better than v1?

blazzy commented 4 years ago

Yeah. v2 would be the right choice. It should have been obvious to me from the path in the request.

haoliu-amp commented 4 years ago

@blazzy Since Identify is basically calling logEvent with some extra logic, I would like that we settle down the fundamental interfaces first. Then I can add it.