Skip to content

fix nested property values#6

Open
neonstalwart wants to merge 2 commits into
dojo:masterfrom
neonstalwart:deep-property
Open

fix nested property values#6
neonstalwart wants to merge 2 commits into
dojo:masterfrom
neonstalwart:deep-property

Conversation

@neonstalwart

Copy link
Copy Markdown

i would write a test but i'm honestly stuck with how to get the tests to run. in lieu of tests, here's a snippet that shows the issue i'm trying to fix with this PR

m = new Model({ object: { foo: 'bar'  }  });
p = m.property('object', 'foo');
console.log(p._get()); // should be 'bar'

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.

1 participant