-
Notifications
You must be signed in to change notification settings - Fork 231
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
Fix custom event arguments #65
Conversation
82b6f36
to
332738d
Compare
+1 waiting for this fix |
Is there any temporary fix until v0.11.2 comes out? Thanks! |
👍 This might be at least merged to a dev branch |
@Tol1 Any chance you could rebase again and update this PR? We're working towards a release soon, and it'd be good to include this fix. |
I'll rebase as soon as I have time |
332738d
to
e7fb159
Compare
Rebased, this should be mergeable now |
👍 Thanks @Tol1 . @CoreSystemDevelopment Could you give this a review as well? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Also probably fixes #89 |
Hi @aaaa0441, I think you put your comment on the wrong issue. This is the crejavascript fork that's being maintained. :-) |
oops, my bad. Thanks, @jlbooker, for keeping this project maintained! =) 👍 |
Unmerged pull request from author of upstream twitter/typeahead.js#1207
Rebased original to be mergeable
Closes #63 and #58