strongloop / strong-arc

StrongLoop Arc has been replaced by API Connect. We recommend Arc users move to the Essentials edition of API Connect. If you have questions, please email reachsl@us.ibm.com.
http://strongloop.com/
Other
114 stars 36 forks source link

tweak un-installed connector error message #373

Open seanbrookes opened 10 years ago

seanbrookes commented 10 years ago

see #346

seanbrookes commented 10 years ago

note the fix for this is in loopback-datasource-juggler https://github.com/strongloop/loopback-datasource-juggler/pull/315

altsang commented 10 years ago

what's confusing in this scenarios is that when you trigger validation by testing connection on one datasource , you're getting back an error on another datasource because of the connector initialization process, we'll deal with this post R2, for now have asked @crandmck to doc

frankcarey commented 10 years ago

This would be helpful.. I've been pulling my hair out trying to understand why I was getting errors from mongo instead of rest connector and I'm assuming this is the cause. Also shouldn't the start fail completely instead of just a warning if a connector is missing?

// Currently (my version of code) it just outputs a warning and drives on.
if (!connector) {
    error = util.format('\nWARNING: LoopBack connector "%s" is not installed ' +
      'as any of the following modules:\n\n %s\n\nTo fix, run:\n\n    npm install %s\n',
      name, names.join('\n'), names[names.length -1]);
  }
bajtos commented 10 years ago

Also shouldn't the start fail completely instead of just a warning if a connector is missing?

Could you please fill an issue in loopback for this?

frankcarey commented 10 years ago

The request was added to https://github.com/strongloop/loopback-datasource-juggler/issues/320