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

fix component attrs inspect #1150

Merged
merged 6 commits into from
Apr 6, 2020
Merged

Conversation

patricklx
Copy link
Collaborator

attrs is an object without prototype, so instanceof Object does not work but typeOf does

attrs is an object without prototype, so instanceof Object does not work but typeOf does
@RobbieTheWagner
Copy link
Member

@chancancode @pzuraq does this look okay to you?

@RobbieTheWagner
Copy link
Member

@chancancode @pzuraq should we merge this?

@pzuraq
Copy link
Contributor

pzuraq commented Apr 5, 2020

Sure, seems fine for now

@RobbieTheWagner RobbieTheWagner merged commit 545e838 into emberjs:master Apr 6, 2020
SYU15 pushed a commit to SYU15/ember-inspector that referenced this pull request Apr 25, 2020
attrs is an object without prototype, so instanceof Object does not work but typeOf does

Co-authored-by: Robert Wagner <rwwagner90@gmail.com>
Co-authored-by: Jerry Nummi <nummi@users.noreply.github.com>
patricklx added a commit to patricklx/ember-inspector that referenced this pull request Sep 19, 2022
attrs is an object without prototype, so instanceof Object does not work but typeOf does

Co-authored-by: Robert Wagner <rwwagner90@gmail.com>
Co-authored-by: Jerry Nummi <nummi@users.noreply.github.com>
@patricklx patricklx deleted the patch-8 branch October 22, 2022 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants