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

Add endpoints for screen recording API for iOS and Android #814

Merged
merged 1 commit into from
Feb 7, 2018

Conversation

mykola-mokhnach
Copy link
Contributor

Change list

Add endpoints for screen recording API for iOS and Android

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

appium/appium-android-driver#303
appium/appium-xcuitest-driver#609

Copy link
Contributor

@TikhomirovSergey TikhomirovSergey left a comment

Choose a reason for hiding this comment

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

@mykola-mokhnach There is the conflict. Could you resolve it?
Will try it ASAP .

@mykola-mokhnach
Copy link
Contributor Author

Rebased it with master

Copy link
Member

@SrinivasanTarget SrinivasanTarget left a comment

Choose a reason for hiding this comment

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

LGTM. Adding some tests will be good still.

@mykola-mokhnach
Copy link
Contributor Author

mykola-mokhnach commented Jan 23, 2018

@SrinivasanTarget I would be happy to add tests, but currently the client interaction with Appium beta is broken, so I cannot even init a session properly. See appium/appium#10055 for more details.

@SrinivasanTarget
Copy link
Member

See appium/appium#10055 for more details.

@mykola-mokhnach Yes just looking at it. I think we need to work on W3C support asap from your other PR.

@mykola-mokhnach
Copy link
Contributor Author

@TikhomirovSergey @SrinivasanTarget The recent server beta has a workaround for w3c issue, so I was able to create and check integration tests for the feature.

Copy link
Member

@SrinivasanTarget SrinivasanTarget left a comment

Choose a reason for hiding this comment

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

Looks Good... Thanks @mykola-mokhnach

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.

None yet

3 participants