search

amplitude / Amplitude-Node

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

v1.0.0 is empty #39

Closed agoldis closed 4 years ago

agoldis commented 4 years ago

Expected Behavior

npm module should contain ... the module

Current Behavior

there's no code in the module

Possible Solution

release the code together with the module

Steps to Reproduce

$ yarn add @amplitude/node
$ cd node_modules/@amplitude/node
$ cat package.json | grep index
  "main": "dist/index.js",
  "module": "esm/index.js",
  "types": "dist/index.d.ts",
$ ls dist/index.js esm/index.js dist/index.d.ts
ls: dist/index.d.ts: No such file or directory
ls: dist/index.js: No such file or directory
ls: esm/index.js: No such file or directory

Environment

kelvin-lu commented 4 years ago

Hi @agoldis ! thanks for flagging. We've identified this as an issue with a recent compilation change. We're looking to release a minor patch soon :)

agoldis commented 4 years ago

That was quick... Good luck!

ZoharBergman commented 4 years ago

Hi @kelvin-lu, I see that you merged a fix but the types and utils packages are still missing image

kelvin-lu commented 4 years ago

Hi @ZoharBergman !

I deployed a hotfix last night and discovered this as well - we're hoping to get this resolved asap, but while more of our engineers are active.

Thank you for reaching out and for your patience - we'll update this asap once we have a new patch out.

kelvin-lu commented 4 years ago

Apologies for the inconvenience - v1.0.2 is now ready for usage on all @amplitude/node+type+utils.