トランジスタ使い始めたばかりの電子工作初心者です。教えてください。

ESP32からDCモーターを制御したいと考えています。
画像の様な回路を考えているのですが、問題無いでしょうか？
また、コンデンサ等でノイズ吸収等した方が良い部分はあるでしょうか？

環境はarduino IDE、ボタンによる割り込みでモーターON、割り込みタイマーでモーターOFFさせたいと考えています。
On信号HIGHでモーター動作開始、自己保持してOff信号HIGHでモーター停止（自己保持解除）の予定です。
https://imgur.com/F19X1Cy