Skip to content

Globalize - #121

Closed
Setogit wants to merge 7 commits into
masterfrom
globalize
Closed

Globalize#121
Setogit wants to merge 7 commits into
masterfrom
globalize

Conversation

@Setogit

@Setogit Setogit commented Jan 13, 2016

Copy link
Copy Markdown
Contributor

connect to strongloop-internal/scrum-nodeops#1157

Comment thread common/models/api.js Outdated

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.

That doesn't make sense to me. This is the documentation string that will show up in the explorer.

  • did you test this works?
  • since almost all the API is defined by JSON files, and they are not translated, won't this be bizarre even if it does work, to have an almost entirely English explorer, with only the remote methods have localized string descriptions?

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.

The unit test passed up to the point master passes locally on my OSX.

strong-mesh-models globalization depends on loopback/explorer globalization strategy which is open at the moment.

@sam-github sam-github assigned Setogit and unassigned sam-github Jan 14, 2016
@Setogit

Setogit commented Jan 16, 2016

Copy link
Copy Markdown
Contributor Author

@sl-node test please

Comment thread bin/sl-meshadm.js Outdated

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.

--help needs protection from translation

@Setogit Setogit assigned sam-github and unassigned Setogit Jan 25, 2016
@Setogit

Setogit commented Jan 25, 2016

Copy link
Copy Markdown
Contributor Author

@sl-node test please

@Setogit

Setogit commented Jan 25, 2016

Copy link
Copy Markdown
Contributor Author

All windows builds failed. The other builds succeeded.

@Setogit
Setogit force-pushed the globalize branch 2 times, most recently from be4eaba to 0c165f0 Compare January 26, 2016 03:56
@Setogit

Setogit commented Jan 26, 2016

Copy link
Copy Markdown
Contributor Author

This requires strongloop/strong-globalize#5

@Setogit

Setogit commented Jan 26, 2016

Copy link
Copy Markdown
Contributor Author

@sl-node test please

Comment thread test/test-meshctl-env.js

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.

@Setogit I don't understand this change.

After globalization, are there no longer quotes around "service 1"?

@sam-github sam-github assigned Setogit and unassigned sam-github Jan 27, 2016
@Setogit Setogit removed their assignment Jul 5, 2016
@dhmlau

dhmlau commented Feb 3, 2020

Copy link
Copy Markdown
Member

Closing as abandoned

@dhmlau dhmlau closed this Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants