-
Notifications
You must be signed in to change notification settings - Fork 292
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
ES6: migrate "localization" #1036
Conversation
if (valueAccessor == null) return; | ||
|
||
let args = []; | ||
let s = valueAccessor; |
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.
better name for s
.. any suggestions?
if (valueAccessor == null) return; | ||
|
||
let args = []; | ||
let s = valueAccessor; |
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.
There is no @param id
nor a @param replace
. I only see @param valueAccessor
.
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.
Next time comment under the right line so it will go away when fixed..
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.
Sure! I just didn't had my glasses on. :P
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.
👓
app/script/localization/Localizer.js
Outdated
} | ||
} | ||
|
||
/* |
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.
JSDOC style /**
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.
heheh :)
I renamed the
strings.coffee
towebapp.js
to have more unique names on Crowdin (mainly because I don't want to lose all the translations there already. I already talked to the customer support of Crowdin and they were very helpful as recently there were some changes on their side and they don't support out of the box*.js
files :) Long story short.. it works!