I would run the wire from transistor emitter back to power supply GND, not Arduino GND, otherwise, looks OK.