amplitude / Amplitude-Node

Server-side Node.js SDK for Amplitude

MIT License

67 stars 20 forks source link

feat: Identify API #46

Closed kelvin-lu closed 3 years ago

kelvin-lu commented 4 years ago

37 # Summary

The work needed to build a way to interface with the Identify API, similar to how the JS SDK currently does. This is a WIP and we plan on adding:

[x] Unit tests
[x] A refactor on the logic in the actual identify client

Note: The tests will fail to build until Identify is added to the NPM registry, but until then I ran the tests locally to ensure they worked.

But wanted to start garnering feedback on design, use, etc.

Resolves Issue #13

Checklist

[ ] Does your PR title have the correct title format?
Does your PR have a breaking change?: