Skip to content

Conversation

@TikhomirovSergey
Copy link
Contributor

Change list

  • Update to selenium 3.7.1
  • Methods beforeAuthentication and afterAuthentication` became deprecared.
  • @mykola-mokhnach was added to the list of developers.

Types of changes

  • No changes in production code.
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details

I can't to update the project to selenium 3.8.x because of mentioned methods. It should be removed safely and users should be warned.

id 'SrinivasanTarget'
};
developer {
name 'Mykola Mokhnach'
Copy link
Member

Choose a reason for hiding this comment

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

👍

* {@link org.openqa.selenium.Alert#authenticateUsing(Credentials)}
* It is deprecated because methods {@link org.openqa.selenium.Alert#setCredentials(Credentials)} and
* {@link org.openqa.selenium.Alert#authenticateUsing(Credentials)} were removed from selenium java client
* at 3.8.0. These listener method is going to be removed too.
Copy link
Contributor

Choose a reason for hiding this comment

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

-> This listener method is going to be removed as well

* {@link org.openqa.selenium.Alert#authenticateUsing(Credentials)}.
* It is deprecated because methods {@link org.openqa.selenium.Alert#setCredentials(Credentials)} and
* {@link org.openqa.selenium.Alert#authenticateUsing(Credentials)} were removed from selenium java client
* at 3.8.0. These listener method is going to be removed too.
Copy link
Contributor

Choose a reason for hiding this comment

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

same as above

Copy link
Contributor

@mykola-mokhnach mykola-mokhnach left a comment

Choose a reason for hiding this comment

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

minor typos only

@appium appium deleted a comment Dec 2, 2017
@TikhomirovSergey TikhomirovSergey merged commit 10ab22a into appium:master Dec 2, 2017
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