Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Isolated scope properties are not available in 'pre' link function when using 'templateUrl' #4193

Closed
wants to merge 1 commit into from

Conversation

trask
Copy link

@trask trask commented Sep 29, 2013

Fix for #2064.

@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@rodyhaddad
Copy link
Contributor

Did you look at #3927 ?

I think it solves the same problem, but lacks the tests to prove it

Isolated scope properties are not available in 'pre' link function when
using 'templateUrl'.

Closes #2064
@trask
Copy link
Author

trask commented Sep 30, 2013

@rodyhaddad Thanks! I just tried the fix from #3927 and you are right it resolves #2064. Great to see it is targeted for 1.2.0-rc3. Closing this PR in favor of #3927.

@IgorMinar
Copy link
Contributor

Thanks, I added similar tests to @vojtajina's PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants