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

added cancel block #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robegamesios
Copy link

Add cancel block for use case where you use a block completion instead of delegate.

@yannickl
Copy link
Owner

Do you really need a cancel block? The completionBlock provides the same purpose.

@robegamesios
Copy link
Author

Hi, can you check, I tried to invoke the completion block trying to cancel the Qr code reader but it does not get fired? Otherwise no need, that's all I was trying to do.

Thanks

Sent from my iPhone

On Jun 10, 2016, at 1:01 PM, Yannick Loriot [email protected] wrote:

Do you really need a cancel block? The completionBlock provides the same purpose.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@yannickl
Copy link
Owner

Sorry for the late,

I've tried and everything works for me:

[vc setCompletionWithBlock:^(NSString *resultAsString) {
   NSLog(@"Completion with result: %@", resultAsString);
}];

And when the cancel button is pushed I got:

Completion with result: (null)

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.

2 participants