RFID RC522 module part for 13.56 Mhz Card

First a google search for “fritzing part rfid rc522”, which is where I always start when considering making a part, turns up an existing part which looks similar to yours:

That said it looks like you have done a good job of it and more parts makers are always useful . Over all it looks fine and very good for a first part.

Peter