Skip to content
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

Pass found elements to the EventListener's afterFind method #6191

Merged
merged 2 commits into from Jul 24, 2018
Merged

Pass found elements to the EventListener's afterFind method #6191

merged 2 commits into from Jul 24, 2018

Conversation

baflQA
Copy link
Contributor

@baflQA baflQA commented Jul 23, 2018

Instead of passing null, pass found element(s) to the afterFindElement(s) method.

Instead of passing null, pass found element(s) to the afterFindElement(s) method.
@baflQA baflQA changed the title Pass found elements to the EventListener's afterFind methods Pass found elements to the EventListener's afterFind method Jul 24, 2018
@shs96c
Copy link
Member

shs96c commented Jul 24, 2018

@baflQA, thank you for this PR. It's very much appreciated! Congratulations on your first commit to the project :)

@shs96c shs96c merged commit b6f307c into SeleniumHQ:master Jul 24, 2018
@baflQA
Copy link
Contributor Author

baflQA commented Jul 24, 2018

I'm sooo happy I could contribute ;) Till the next one @shs96c !

@baflQA baflQA deleted the patch-1 branch July 24, 2018 10:43
grigaman pushed a commit to grigaman/selenium that referenced this pull request Sep 20, 2018
…HQ#6191)

Instead of passing null, pass found element(s) to the afterFindElement(s) method.
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.

None yet

2 participants