-
-
Notifications
You must be signed in to change notification settings - Fork 413
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
feat: Add mobile commands to cover new simctl features #1162
Conversation
lib/commands/appearance.js
Outdated
return this.isSimulator() | ||
? await this.opts.device.setAppearance(style) | ||
// ugly workaround for real devices | ||
: await this.mobileSiriCommand({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can the mobileSiriCommand
workaround be used on simulators with Xcode < 11.4 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we can try it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for jumping on this @mykola-mokhnach !
@@ -84,6 +84,10 @@ extensions.executeMobile = async function executeMobile (mobileCommand, opts = { | |||
clearKeychains: 'mobileClearKeychains', | |||
|
|||
getPermission: 'mobileGetPermission', | |||
setPermission: 'mobileSetPermissions', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about changePermissions
like Android?
(I also thought to keep setPermissions
for iOS since the parameter was different. So, it is fine to go with current name, too.)
No description provided.