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

Alert view support for iOS [redacted] #20

Merged
merged 2 commits into from Aug 25, 2013

Conversation

olarivain
Copy link
Contributor

Hey Karl,

After a few weeks of procrastination, it's finally here: support for tapping alert views in iOS whose version shalt not be named.

As usual, patches are WTF public license, just have an extra requirement this time: mention the patch was made by myself, on behalf of OpenTable (we're trying to get our name out in the iOS community :) )

Anyway, let me know if anything's wrong with the patch.

@olarivain
Copy link
Contributor Author

And of course, I had overlooked the fact that calabash doesn't use ARC. added the proper autoreleases in the right place.

krukow added a commit that referenced this pull request Aug 25, 2013
Alert view support for iOS [redacted]
@krukow krukow merged commit 872b823 into calabash:calabash-ios-server Aug 25, 2013
@krukow
Copy link
Member

krukow commented Aug 25, 2013

Thanks @olarivain of OpenTable! I'll include this in the release notes.

@ghost
Copy link

ghost commented Sep 6, 2013

Thanks! Nice work

@krukow
Copy link
Member

krukow commented Sep 7, 2013

Merged and released with 0.9.152

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

2 participants