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

Refactor eyesswitchto #29

Merged
merged 14 commits into from
Dec 4, 2018
Merged

Refactor eyesswitchto #29

merged 14 commits into from
Dec 4, 2018

Conversation

eirenik0
Copy link
Contributor

@eirenik0 eirenik0 commented Nov 7, 2018

No description provided.

@eirenik0 eirenik0 changed the base branch from master to develop November 7, 2018 13:04
@eirenik0 eirenik0 added this to To do in Ongoing tasks Nov 7, 2018
@eirenik0 eirenik0 moved this from To do to In progress in Ongoing tasks Nov 7, 2018
@eirenik0 eirenik0 force-pushed the refactor-eyesswitchto branch 2 times, most recently from 66c3a72 to 06525f1 Compare November 12, 2018 09:52
@isaulv
Copy link

isaulv commented Nov 14, 2018

Does this branch fix issues with working with iframes?

@eirenik0
Copy link
Contributor Author

@Dude-x which issue do you mean?

@isaulv
Copy link

isaulv commented Nov 16, 2018

@Infernion It's not a bug per se, but given a locator (By, String) it will retrieve it only once and then use that element multiple times. We get StaleElementReferenceExceptions occasionally due to some minor modification of the DOM.

@eirenik0
Copy link
Contributor Author

eirenik0 commented Nov 17, 2018

@Dude-x this issue related to switching between frames. This will be fixed but not sure in current or in the following PR.

@eirenik0 eirenik0 force-pushed the develop branch 2 times, most recently from cd3fc54 to 5457091 Compare November 27, 2018 20:32
@eirenik0 eirenik0 force-pushed the develop branch 2 times, most recently from ffc7ed1 to 7683524 Compare November 28, 2018 15:14
@eirenik0 eirenik0 merged commit d65e3d6 into develop Dec 4, 2018
Ongoing tasks automation moved this from In progress to Done Dec 4, 2018
@eirenik0 eirenik0 deleted the refactor-eyesswitchto branch December 4, 2018 14:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Ongoing tasks
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants