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.

Sign up for GitHub

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

Jump to bottom

fix(ios): volume event handling #11216

Merged
merged 2 commits into from Sep 20, 2019
Merged

fix(ios): volume event handling #11216

merged 2 commits into from Sep 20, 2019

Conversation

vijaysingh-axway
Copy link
Contributor

@build build added this to the 8.3.0 milestone Sep 14, 2019
@build build requested a review from a team September 14, 2019 00:32
@build
Copy link
Contributor

build commented Sep 14, 2019 •
edited

Messages
📖 👍 Hey!, You deleted more code than you added. That's awesome!
📖

💾 Here's the generated SDK zipfile.
📖

✅ All tests are passing
Nice one! All 3898 tests are passing.
(There are 475 skipped tests not included in that total)
📖 ✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.

Generated by 🚫 dangerJS against ca3b99c

@keerthi1032
Copy link
Contributor

FR passed.Volume event triggered and printed in console on pressing volume button.

Test Environment:
Operating System
Name = Mac OS X
Version = 10.14.6
Architecture = 64bit
Node.js
Node.js Version = 10.16.2
npm Version = 6.9.0
Titanium CLI
CLI Version = 5.2.1
Titanium SDK
SDK Version = local sdk 8.3.0.v20190913171045 and 8.2.0.v20190917160001
Device -iPhone XR iOS 13.1,iphone x iOS 13,iphone 6s iOS 12

@keerthi1032 keerthi1032 merged commit c697822 into tidev:master Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@janvennemann janvennemann janvennemann approved these changes

Assignees
No one assigned
Labels
bug in-qe-testing 🕵 ios no tests
Projects
None yet
Milestone
8.3.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vijaysingh-axway @build @keerthi1032 @janvennemann