Stop enforcing that every model has an ID - not a big deal anymore because the client can’t save!

packages/client-app/src/flux/attributes/attribute-collection.es6

@@ -76,16 +76,10 @@ export default class AttributeCollection extends Attribute {
 
       if (this.ItemClass.prototype.fromJSON) {
         const obj = new this.ItemClass();
-        // Important: if no ids are in the JSON, don't make them up
-        // randomly.  This causes an object to be "different" each time it's
-        // de-serialized even if it's actually the same, makes React
-        // components re-render!
-        obj.id = undefined;
         obj.fromJSON(objJSON);
         objs.push(obj);
       } else {
         const obj = new this.ItemClass(objJSON);
-        obj.id = undefined;
         objs.push(obj);
       }
     }

packages/client-app/src/flux/models/model.coffee

@@ -38,8 +38,6 @@ class Model
     for key in Object.keys(@constructor.attributes)
       continue unless values[key]?
       @[key] = values[key]
-
-    @id ?= Utils.generateTempId()
     @
 
   isSavedRemotely: ->

packages/client-app/src/flux/models/mutable-query-result-set.es6

@@ -94,7 +94,6 @@ export default class MutableQueryResultSet extends QueryResultSet {
     }
 
     this._modelsHash[item.id] = item;
-    this._idToIndexHash = null;
   }
 
   removeModelAtOffset(item, offset) {