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

[TIMOB-24349] iOS: Fix both AlertDialog and OptionDialog to fire events properly #9803

Merged
merged 3 commits into from Apr 12, 2018

Conversation

hansemannn
Copy link
Collaborator

@hansemannn hansemannn commented Feb 6, 2018

JIRA: https://jira.appcelerator.org/browse/TIMOB-24349

no tests because it's a UI related change.

@build
Copy link
Contributor

build commented Feb 6, 2018

Messages
📖

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

📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR passed.

@build build added the ios label Feb 14, 2018
@hansemannn hansemannn modified the milestones: 7.1.0, 7.2.0 Feb 16, 2018
@build build added the ios label Apr 9, 2018
@ewieberappc
Copy link
Contributor

FR Passed. All alert/click events were fired and heard correctly, including those in the background. Tested using the provided samples, modified versions to check additional variants, and the alert dialog suite.

@ewieberappc ewieberappc merged commit 3432de2 into tidev:master Apr 12, 2018
build pushed a commit to garymathews/titanium_mobile that referenced this pull request Apr 23, 2018
…ts properly (tidev#9803)

* [TIMOB-24349] Fix alert dialog event from not firing

* [TIMOB-24349] Fix option dialog as well
@sgtcoolguy sgtcoolguy modified the milestones: 7.2.0, 7.3.0 May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants