Skip to content

Conversation

@v-escobar
Copy link

This function receives the language name ( how appears in the language list) and it installs the language if exists. This function also refresh the language list to ensure that the language is there.

Copy link
Member

Choose a reason for hiding this comment

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

I think this xpath can be modified and made more readable, Is it not possible?

Copy link
Member

Choose a reason for hiding this comment

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

by the way

//* is not the correct way, it will make Codeception search through all the elements, and make the tests slower
//div, //form //p the element has to be inside some of them 

@puneet0191
Copy link
Member

Thanks for PR! 👍 @vistiyos

Copy link
Contributor

Choose a reason for hiding this comment

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

not important but more performant: $I->click(['id' => "cb0"]);

Copy link
Author

Choose a reason for hiding this comment

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

@javigomez I have added that in the new commit

@javigomez
Copy link
Contributor

Thanks for the improvement @vistiyos ^_^

@javigomez javigomez closed this in 359e48f Jul 22, 2015
@javigomez
Copy link
Contributor

I have merged at 359e48f resolving the recent conflicts.

@puneet0191 leaving it to you for improving the paths in future pulls

@puneet0191
Copy link
Member

yeap thanks 👍

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