Call the BlockEntity#setChanged method after updating nbt data so minecraft 

knows that your block is updated.

​

One other thing to notice is that serializeNBT and deserializeNBT methods 

from BlockEntity class are internal methods, instead use 

BlockEntity#saveAdditional and BlockEntity#load