Skip to content

Add globalization#24

Merged
superkhau merged 1 commit intomasterfrom
globalization
Aug 9, 2016
Merged

Add globalization#24
superkhau merged 1 commit intomasterfrom
globalization

Conversation

@superkhau
Copy link
Copy Markdown
Contributor

@superkhau superkhau commented Aug 8, 2016

@0candy PTAL

Comment thread example/example.js
@@ -33,7 +33,7 @@ db.autoupdate('User', function(err) {
});
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need this line at the beginning of file
var g = require('strong-globalize')();

@0candy 0candy assigned superkhau and unassigned 0candy Aug 8, 2016
@superkhau superkhau force-pushed the globalization branch 3 times, most recently from 1975ff9 to b6d3bab Compare August 8, 2016 20:58
@superkhau
Copy link
Copy Markdown
Contributor Author

@0candy PTAL again

Comment thread intl/en/messages.json
@@ -0,0 +1,3 @@
{
"5bdcd0e36db5b1100819f9aa25e9bdf0": "Cloudant {{regex}} syntax does not support global"
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

weird, how come still doesn't show the above string example complete?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that is strange, checking now, i'll try rerunning linter

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

its good now, reruning global slt-globalize -e did it now. probably because of the bug this morning during the demo with ./node_modules...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@0candy PTAL again

Comment thread package.json Outdated
"lodash": "^3.10.1",
"loopback-connector": "^2.3.0"
"loopback-connector": "^2.3.0",
"strong-globalize": "^2.6.5"
Copy link
Copy Markdown
Contributor

@0candy 0candy Aug 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we put ^2.6.6 as 2.6.5 has a bug? Probably not needed, up to you

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caret will pick up 2.6.6 anyways, but I'll update it now while we have the chance.

@0candy
Copy link
Copy Markdown
Contributor

0candy commented Aug 9, 2016

LGTM

@superkhau
Copy link
Copy Markdown
Contributor Author

Merging. Unrelated linting failures on CI.

@superkhau superkhau merged commit 34b88e1 into master Aug 9, 2016
@superkhau superkhau deleted the globalization branch August 9, 2016 21:07
@superkhau superkhau removed the #review label Aug 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants