One-com / knockout-projections

Knockout.js observable arrays get smarter
5 stars 2 forks source link

UniqueIndexBy does not handle reindexing on another field that conficts with the current index #2

Open sunesimonsen opened 9 years ago

sunesimonsen commented 9 years ago

I create a test case the proves this problem: handles reindexing on another field The test is skipped as it fails right now.

We should you reindex completely when we see a conflict like the sortBy projection. If the reindex fails we should still fail.