Uh oh!
There was an error while loading. Please reload this page.
Add battery information accessors - #882
Conversation
mykola-mokhnach
commented
Apr 30, 2018
@KazuCocoa FYI |
| public BatteryStatus getStatus() { | ||
| final int status = ((Long) getInput().get("status")).intValue(); | ||
| switch (status) { | ||
| case 2: |
There was a problem hiding this comment.
I think it would be better to move these magic numbers to enum.
There was a problem hiding this comment.
these are used only once in this particular place. Creating a separate enum for them would be an overkill.
TikhomirovSergey
commented
May 2, 2018
@mykola-mokhnach There is the conflict. Could you resolve it? |
… into battery # Conflicts: # src/main/java/io/appium/java_client/ios/IOSDriver.java
mykola-mokhnach
commented
May 2, 2018
Resolved |
TikhomirovSergey
commented
May 5, 2018
@mykola-mokhnach Going to run Android tests today. |
mykola-mokhnach
commented
May 6, 2018
@TikhomirovSergey Android is still not merged. There are problems with server publishing. I'll add a comment when it's done |
TikhomirovSergey
commented
May 6, 2018
@mykola-mokhnach Ok. Maybe there is sense to publish 6.0.0 without this change? How about 6.0.1? |
mykola-mokhnach
commented
May 7, 2018
@TikhomirovSergey No problems |
mykola-mokhnach
commented
May 7, 2018
The Pr for uia2 driver has been merged, so this PR can be merged as well. |
SrinivasanTarget
commented
May 7, 2018
@TikhomirovSergey We are good to release 6.0.0 with this now :) |
Change list
Depends on
appium/appium-uiautomator2-driver#164
appium/appium-xcuitest-driver#661
Types of changes