search

jjhoughton / napi-ldap

MIT License
7 stars 5 forks source link

Docmentation says to require ldap-client #9

Closed cout closed 3 years ago

cout commented 3 years ago

README.md says that to use napi-ldap, I should require ldap-client:

var LDAP = require('ldap-client');

Should this say napi-ldap instead?

jjhoughton commented 3 years ago

Sorry that's a really good point. It's a rewrite of this project https://github.com/jeremycx/node-LDAP I guess i forgot to update the documentation, I will do so shortly when i've finished work. You are correct it should be const LDAP = require("napi-ldap");

jjhoughton commented 3 years ago

Thanks for raising it.

jjhoughton commented 3 years ago

Closing as the documentation has been updated in v1.0.2.