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 13989 map long click #13

Merged
merged 4 commits into from Oct 5, 2013

Conversation

salachi
Copy link
Contributor

@salachi salachi commented Sep 22, 2013

Added support for longClick event

@chermus
Copy link

chermus commented Sep 24, 2013

Please merge this.

@chermus
Copy link

chermus commented Sep 26, 2013

Can you merge this today?

@hieupham007
Copy link
Contributor

Sunil, can you add documentation for this new event?

@chermus
Copy link

chermus commented Sep 30, 2013

And now we wait another month :-(

@hieupham007
Copy link
Contributor

Please merge with master and update the changelog (this is because the current version of map is recently bumped to 2.1.3). Please also include a built zip of the module. You can refer to https://github.com/appcelerator-modules/ti.map/pull/4/files as an example.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Looks good, just need to modify changelog and add a built zip.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Request accepted

jonalter added a commit that referenced this pull request Oct 5, 2013
@jonalter jonalter merged commit 8484f97 into tidev:master Oct 5, 2013
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

4 participants