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

Fixed function overwrite #1309

Merged
merged 1 commit into from Nov 7, 2014

Conversation

Projects
None yet
2 participants
@ivan-nikitovic
Contributor

ivan-nikitovic commented Nov 5, 2014

In case function is already implemented in extended model generator based on resource property overwrites it which should not be the case.

Fixed function overwrite
In case function is already implemented in extended model generator based on resource property overwrites it which should not be the case.

@daffl daffl added this to the 2.1.4 milestone Nov 6, 2014

@daffl

This comment has been minimized.

Show comment
Hide comment
@daffl

daffl Nov 6, 2014

Contributor

Thank you! I think #1272 and #1246 are related. I will add some tests and verify that this closes it.

Contributor

daffl commented Nov 6, 2014

Thank you! I think #1272 and #1246 are related. I will add some tests and verify that this closes it.

@daffl daffl self-assigned this Nov 7, 2014

daffl added a commit that referenced this pull request Nov 7, 2014

@daffl daffl merged commit 8fc8a9d into canjs:master Nov 7, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

daffl added a commit that referenced this pull request Nov 7, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment