-
Notifications
You must be signed in to change notification settings - Fork 33
Add globalization #299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add globalization #299
Conversation
22e634e to
0b3bf18
Compare
3adb844 to
1eb6839
Compare
|
@jannyHou or @superkhau PTAL. Thanks. |
| return cb( | ||
| new Error('Cannot connect to the data source.' + | ||
| ' Ensure the configuration is valid and the connector is installed.')); | ||
| new Error(g.t('Cannot connect to the data source.' + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
g.f
|
@0candy Only some reminders that change |
9bdc5d5 to
ade210a
Compare
| }, | ||
| function createUser(next) { | ||
| console.log('Creating user %s with password %s', USER, PASSWORD); | ||
| console.log(g.f('Creating user %s with password %s', USER, PASSWORD)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see inconsistency between using console.log or g.log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 We should allow either but probably recommend g.log in globalization.md.
strongloop/loopback#2422