-
Notifications
You must be signed in to change notification settings - Fork 42
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
Can't patch Backbone model Of typescript app #227
Comments
Honestly I'm not sure if the inspector supports typescript. There's an es6 fix on minor for example. Happy to pair tomorrow or next week and figure it out. Come to our gitter room, link is on the readme. |
How can I tell if this fix has made it into the extension I'm using? I have version 0.5.4 if that helps. I am hitting something that looks like a similar issue, but I don't really know how to tell. |
hey @gwynjudd mind running it locally and letting me know what you see https://github.com/marionettejs/marionette.inspector/blob/master/docs/developing_locally.md |
I get the same issue as I was seeing before (this is in backbone.marionette.js):
On the line I marked above,
In the stack before this is a lot of function calls from the extension itself. Here is the top of the stack: |
Application is important because that's how the views are found. Try checking __agent to see if the views are saved. Thanks for the details , we'll get to the bottom of tyis |
@jasonLaster - what do I look at in __agent to see the views? Here is the top bit of __agent in the developer tools: |
Hi, sorry this was made private recently __agent.getAppComponentsInfo('View') |
Hmm try looking at the region tree. __agent.appObserver.regionTree() I think. |
Getting this error when attempting to open up Marionette Inspector:
The model in question:
When I log
this
inside theget query()
, I get this.This looks to be an issue with getters and setters. Does Marionette Inspector support these? Is it due to the way Typescript is compiling these?
The text was updated successfully, but these errors were encountered: