Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

refactor(ng-repeat): refactor fix for #1015 #1018

Closed
wants to merge 1 commit into from

Conversation

vicb
Copy link
Contributor

@vicb vicb commented May 8, 2014

This is my proposal to fix #1015.
#1017 has already been merged but I think this set of change is easier to understand and more efficient.

See the changes vs the original (ie bugged) version.

Also if #774 eventually gets fixed, this version would be easier to deal with.

/cc @chirayuk

@jbdeboer
Copy link
Contributor

@chirayuk Could you review this change?

@vicb
Copy link
Contributor Author

vicb commented May 20, 2014

@chirayuk any objection for this to be merged ? I'd like to merge it today

@mvuksano mvuksano assigned vicb and unassigned chirayuk and vicb May 26, 2014
@vicb
Copy link
Contributor Author

vicb commented Jun 4, 2014

I'm going to merge this. It helps when working on ng-repeat issues

@vicb
Copy link
Contributor Author

vicb commented Jun 4, 2014

closed via f6187ae

@vicb vicb closed this Jun 4, 2014
@vicb vicb deleted the 0508-ngrepeat branch June 4, 2014 16:38
vicb added a commit to vicb/angular.dart that referenced this pull request Aug 26, 2014
This re-introduce the PR dart-archive#1018: "refactor(ng-repeat): refactor fix for
dart-archive#1015" which had been reverted.
vicb added a commit that referenced this pull request Aug 26, 2014
This re-introduce the PR #1018: "refactor(ng-repeat): refactor fix for
#1015" which had been reverted.
vicb added a commit that referenced this pull request Aug 26, 2014
This re-introduce the PR #1018: "refactor(ng-repeat): refactor fix for
#1015" which had been reverted.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

ng-repeat get corrupted when iterating over a null
3 participants