dotnet-client icon indicating copy to clipboard operation
dotnet-client copied to clipboard

[Feat]: replace non-w3c commands with mobile:, and add deprecation message if needed

Open KazuCocoa opened this issue 2 years ago • 8 comments

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

See https://github.com/appium/java-client/pull/1883

We're going to move non-w3c endpoint call to mobile (macos for mac2 driver etc) command, eventually, in the server side. Clients also want to follow the way.

Describe the solution you'd like A clear and concise description of what you want to happen.

  • Remove non-w3c endpoint call, instead calls mobile extension command properly
  • Mark non-w3c endpoint call as deprecate

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

KazuCocoa avatar Apr 14 '23 06:04 KazuCocoa

@KazuCocoa has a mobile extension been created for every non-w3c command? Or just the app management endpoints?

laolubenson avatar Apr 14 '23 06:04 laolubenson

Maybe already exists: e.g. https://github.com/appium/appium-uiautomator2-driver

Potentially some are not, then we should add them in the server side

KazuCocoa avatar Apr 14 '23 06:04 KazuCocoa

Each driver repository may have the list in the documentation

KazuCocoa avatar Apr 14 '23 06:04 KazuCocoa

@laolubenson are you taking this? or should I add it to my TO-DO?

Dor-bl avatar May 05 '23 18:05 Dor-bl

@laolubenson are you taking this? or should I add it to my TO-DO?

@Dor-bl I'll pick this up.

laolubenson avatar May 05 '23 19:05 laolubenson

@KazuCocoa, do you mean deprecating this region at first? https://github.com/appium/dotnet-client/blob/main/src/Appium.Net/Appium/AppiumDriverCommand.cs#L19C53-L19C53

Dor-bl avatar Sep 28 '23 09:09 Dor-bl

@KazuCocoa, do you mean deprecating this region at first? https://github.com/appium/dotnet-client/blob/main/src/Appium.Net/Appium/AppiumDriverCommand.cs#L19C53-L19C53

@KazuCocoa, do you have any input here?

Dor-bl avatar Apr 12 '24 17:04 Dor-bl

yea, it looks correct. Perhaps not all commands already have mobile:, or still need to keep the command, but basically yes.

At least the file's commands will be via mobile: eventually or can simply drop as it no longer works. Some could keep as current deprecation comments for now as dotnet client v5, and can drop them as client v6 later.

KazuCocoa avatar Apr 12 '24 17:04 KazuCocoa

@Dor-bl @KazuCocoa

Can I change the mobile prefix of the commands in AppiumDriverCommand based on appium/java-client#1883 and https://github.com/appium/appium-uiautomator2-driver?

namilkimfree avatar Feb 28 '25 08:02 namilkimfree

@Dor-bl @KazuCocoa

Can I change the mobile prefix of the commands in AppiumDriverCommand based on appium/java-client#1883 and https://github.com/appium/appium-uiautomator2-driver?

@namilkimfree It's not that simple since the commands fromAppiumDriverCommand are also been used on xcuitest driver. It will require a more clinical approach. I suggest you start with one command at a time while making sure nothing breaks in the process.

Dor-bl avatar Feb 28 '25 09:02 Dor-bl

@Dor-bl

I will check the driver code and server

namilkimfree avatar Mar 05 '25 00:03 namilkimfree

https://github.com/appium/dotnet-client/pull/939

KazuCocoa avatar May 31 '25 08:05 KazuCocoa