Hi, a couple suggestions here, if I may - Since the building still exists we'll want to keep some sort of building=* tag. Also, (I had to look this up) - The wiki recommends to avoid having the old name on a shop=vacant feature. https://wiki.openstreetmap.org/wiki/Tag:shop=vacant?uselang=en#How_to_map

I went ahead and fixed those. Let me know if you disagree (I'm open to discussion, of course!) or have questions! Thanks for your contributions.