Skip to content

fix: components having attributes that start with on#1530

Merged
marvinhagemeister merged 3 commits into
masterfrom
fix/debug
Apr 12, 2019
Merged

fix: components having attributes that start with on#1530
marvinhagemeister merged 3 commits into
masterfrom
fix/debug

Conversation

@JoviDeCroock

@JoviDeCroock JoviDeCroock commented Apr 11, 2019

Copy link
Copy Markdown
Member

Fixes: #1504 (comment)

There were scenario's that components had properties starting with 'on' for example online.

@coveralls

coveralls commented Apr 11, 2019

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling b9e269f on fix/debug into 83c76bd on master.

Comment thread debug/src/debug.js Outdated
Comment thread debug/src/debug.js Outdated

@marvinhagemeister marvinhagemeister left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Much appreciated 👍 🎉

@marvinhagemeister marvinhagemeister merged commit ef01a85 into master Apr 12, 2019
@marvinhagemeister marvinhagemeister deleted the fix/debug branch April 12, 2019 08:27
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.

[10.0.0-alpha.3] Component's onClick property should be a function

3 participants