Hello.
Image tag should be just a url to an image, no html tags or javascript.
name= tag should be just the name of that feature. brand= tag should be separate from name=
phone= tag should always contain country code (+1 for US) and separated only by spaces or - not any other characters like ().

For a business with multiple locations it's also useful to add branch= tag with the name of the given location to easier distinguish between them.

If a business occupies the whole building(such as the case here), it's ok to add the business' tags to the building itself, rather than a separate node like you did. The way you mapped it is not wrong, but another option.

iD editor comes with many preset for different business, this Extra Space Storage is one of them. You could use it to automatically pre-fill many of the common tags between all these business.
            ---
            
            Published using OSMCha: https://osmcha.org/changesets/134278509
            