Making production data accessible to developers via masking
Is it a reasonable approach?


Yes.  That's what the (removed) "Static Data Masking" component in SSMS did.  A simple way to implement this is to copy data from production using an identity for which Dynamic Data Masking is enforced on the production database, using the Copy Database Wizard or a custom script.