Add idb-connector to Dependencies in Package.json

[abmusse]

Original report by me.

---

Package.json Should list idb-connector as a dependancy because it is required on line 50 of istroedp.js.

#!js
  db = require('idb-connector');

That way if someone npm installs itoolkit without already having idb-connector it will work.

[abmusse]

Original comment by Aaron Bartell (Bitbucket: aaronbartell, GitHub: aaronbartell).

---

I think the reason we originally didn't put it as a dependency is because the iToolkit can be used in an HTTP scenario where idb-connector isn't required. With that said, it is much more often the case that the iToolkit is used in scenarios where idb-connector is required so I am in favor of this change.

[abmusse]

Original comment by Xu Meng (Bitbucket: mengxumx, GitHub: dmabupt).

---

@ThePrez what's your opinion?

[kadler]

Seems like this would be sufficient: https://docs.npmjs.com/files/package.json#optionaldependencies

[abmusse]

Seems like this would be sufficient: https://docs.npmjs.com/files/package.json#optionaldependencies

So on non ibm i systems itoolkit will still install properly even though idb-connector wont install.

[abmusse]

Moved idb-connector to optional deps in 85a2e54