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

Protocol fix adding contactPicker and exposing didBegin, didEndEditing #49

Merged
merged 3 commits into from
Jan 14, 2016

Conversation

eliburke
Copy link

This PR updates the protocol definition to match what @tomaskraina suggested in #33 and #34 (plus didBeginEditing, which I use to determine which set of values to load into the table for two different contactPickers).

@eliburke
Copy link
Author

Sorry I know I've dirtied up this patch a bit. I was getting quite confused by all of the "textView"s that actually referred to a textField (I'm guessing your underlying implementation changed).

I know this completely breaks any kind of backwards compatibility.. if you want, I'd be willing to re-add the old protocol methods as deprecated.

@dtaitz
Copy link

dtaitz commented Oct 13, 2015

This would be a great addition to the project.

@tristanhimmelman tristanhimmelman merged commit a299f80 into tristanhimmelman:master Jan 14, 2016
@tristanhimmelman
Copy link
Owner

Hi @eliburke, sorry for the loooong delay in getting to this. I haven't been updating this project much lately.

Thanks for the changes to the protocol, they are definitely an improvement.

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.

3 participants