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

refactor!: Drop APIs related to io.appium.settings interactions #716

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

mykola-mokhnach
Copy link
Contributor

@mykola-mokhnach mykola-mokhnach commented Jan 11, 2024

BREAKING CHANGE: Removed isSettingsAppServiceRunningInForeground method
BREAKING CHANGE: Removed requireRunningSettingsApp method
BREAKING CHANGE: Removed setAnimationState method
BREAKING CHANGE: Removed setDeviceSysLocaleViaSettingApp method
BREAKING CHANGE: Removed setGeoLocation method
BREAKING CHANGE: Removed getGeoLocation method
BREAKING CHANGE: Removed refreshGeoLocationCache method
BREAKING CHANGE: Removed getClipboard method
BREAKING CHANGE: Removed getNotifications method
BREAKING CHANGE: Removed getSmsList method
BREAKING CHANGE: Removed typeUnicode method
BREAKING CHANGE: Removed scanMedia method
BREAKING CHANGE: Removed setDeviceLocale method
BREAKING CHANGE: Removed setDeviceLanguageCountry method
BREAKING CHANGE: Removed setWifiAndData method
BREAKING CHANGE: setWifiState method only supports switching Wi-Fi state on real devices since API 30 (emulator support is unaffected)
BREAKING CHANGE: setDataState method only supports switching data state on real devices since API 30 (emulator support is unaffected)

All utilities that implicitly rely on the presence of the io.appium.settings application have been removed from this package and moved to the io.appium.settings module.

Copy link
Member

@KazuCocoa KazuCocoa left a comment

Choose a reason for hiding this comment

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

lg overall, retrying CI to be happy

@mykola-mokhnach mykola-mokhnach merged commit dcfe5a6 into appium:master Jan 12, 2024
10 of 11 checks passed
@mykola-mokhnach mykola-mokhnach deleted the rm_settings branch January 12, 2024 21:53
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