Hi, welcome to OpenStreetMap and many thanks for updating the house details here.

You asked for a review and it looks fine to me.

The only thing which you may want to change is the addr:housenumber tag. There are several ways to tag a building with multiple house numbers (link below), but as another local mapper has already been using a comma, setting it to "29,31" may be the best choice.

https://wiki.openstreetmap.org/wiki/Addresses#Buildings_with_multiple_house_numbers