posva / vuexfire

Check
https://github.com/vuejs/vuefire
MIT License
558 stars 49 forks source link

Fix main/module/browser entry points #190

Closed mattdsteele closed 6 years ago

mattdsteele commented 6 years ago

Hi! I attempted to update from vuexfire@3.0.0-alpha.5 to alpha.8 and began receiving this error in my project:

This dependency was not found:

* vuexfire in ./src/store/index.ts, ./src/store/login/actions.ts

It appears starting with alpha.6, the files in the dist/ folder were named differently (ex: vuexfire.cjs.js became vuexfire.common.js), which breaks tools like Webpack and TypeScript.

This updates the entry points to reflect the files as they're published in NPM.

posva commented 6 years ago

hey, could you do the PR at https://github.com/vuejs/vuefire/tree/lerna, it's now managed there, thanks!

mattdsteele commented 6 years ago

Done! https://github.com/vuejs/vuefire/pull/204