Skip to content

Conversation

@andrewplummer
Copy link
Contributor

This is the same change as #1140, minus a few back and forth commits that were reverted, and with a simpler unit test, as per @gnarf37's request.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double quotes

@gnarf
Copy link
Member

gnarf commented Jan 31, 2013

gnarf pushed a commit that referenced this pull request Jan 31, 2013


Squashed commit of the following:

commit 5935a36
Author: Andrew Plummer <[email protected]>
Date:   Fri Feb 1 02:40:42 2013 +0900

    Using "hasOwnProperty" to check for direct properties "type" and
    "namespace" on events before triggering.
@gnarf gnarf closed this in d654914 Jan 31, 2013
@gnarf
Copy link
Member

gnarf commented Jan 31, 2013

Landed in 1.9-stable and master.

@andrewplummer
Copy link
Contributor Author

@gnarf37 thank you!

Sorry, force of habit.

@gnarf
Copy link
Member

gnarf commented Jan 31, 2013

⭐ Thanks for being persistant, and listening, and above all the contribution. ⭐

👍

@andrewplummer
Copy link
Contributor Author

Of course! In a sense I'm asking for a lot, I realize that.
Here's hoping there's no more.

mescoda pushed a commit to mescoda/jquery that referenced this pull request Nov 4, 2014
…rygh-1153

Squashed commit of the following:

commit 5935a36
Author: Andrew Plummer <[email protected]>
Date:   Fri Feb 1 02:40:42 2013 +0900

    Using "hasOwnProperty" to check for direct properties "type" and
    "namespace" on events before triggering.
@lock lock bot locked as resolved and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants