Using `.key` in track-by throws error about vuefire

[yyx990803]

Ah, a change recently introduced in Vue 1.0.21 caused this error... it is now fixed in the dev branch, and will be out in the next release (soon). In the meanwhile you can either downgrade to 1.0.20, or build Vue core from the dev branch yourself.

Sorry about that!

[humanismusic]

I'm seeing the same problem so I downgraded to 1.0.20 as suggested. This stopped the duplicate warning appearing however now duplicate data shows whenever I update Firebase.

I'm using Sortable drag and drop and when I move an item from one list to another I use a directive triggered on dragend to set the items list property to the name of the new list (I use this list name as a filter in my v-for template). This was working before the switch to Vuefire. I'd update the item property passed in the directive:

v-updatelist="task.list"
destList = $(this.el).closest("ul").attr("data-list"); //get destination list
this.set(destList); //set list property of this item to the destination list

then update Firebase:

baseTasks.child(draggedID).update({ list: destType, }, onComplete);

Now I've removed updating the Vue data (this.set) as Vuefire should handle the binding.
On dragend Firebase updates correctly but now duplicates appear on screen. I've checked the Vue data and the duplicates are not there, so issue appears to be around the v-for.

I don't see any mention of using update in your docs so wondering whether this is supported or if I'm simply going about it in the wrong way?

[yyx990803]

@weislanes do you have a way to reproduce the issue you described? If yes please open a separate issue with the repro.

[humanismusic]

@yyx990803 I'm currently trying to reproduce and looks like it may be an issue with the layout of slots in my app structure. Will raise separately if the issue persists after I change a few things, cheers. 👍

[jwngr]

Closing since this issue is a bit stale and it's not clear it is an issue with the actual VueFire library. If you are still having issues, let us know and we can re-open.