Skip to content
This repository has been archived by the owner on Feb 17, 2021. It is now read-only.

dont ignore i18n object for specific step in tour #170

Closed
wants to merge 6 commits into from

Conversation

nezaidu
Copy link

@nezaidu nezaidu commented Apr 24, 2015

No description provided.

@@ -756,11 +756,11 @@

// Determine label for next button
if(isLast){
nextBtnText = utils.getI18NString('doneBtn');
nextBtnText = (step.i18n && step.i18n["doneBtn"]) || utils.getI18NString('doneBtn');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use dot syntax here instead of brackets... step.i18n && step.i18n.doneBtn.

(+ other instances)

@timlindvall
Copy link
Collaborator

Thanks for the pull request, @nezaidu! It looks like the master branch of your fork has a couple of additional commits unrelated to this pull request that are getting included automatically. Could you create a new branch using the changes in your first commit and push that to your fork on GitHub? Hopefully we can update this PR to point to the new branch, but if not you'll need to create a new PR using the new branch.

We should also cover this under our unit tests to ensure we don't have regression on this feature in the future. If you have Grunt (http://gruntjs.com/) already installed locally, you can use grunt test to run the test suite locally.

@timlindvall
Copy link
Collaborator

Closing this PR due to inactivity. We'll continue to track this issue on #169 and address this when we clean up how we handle configs in version 0.3.0.

@timlindvall timlindvall closed this Jun 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants