This is my first PCB, and I would like someone to look over the PCB design I currently have and let me know if there are any obvious errors that have been made.
Thanks!

 

hi DS,
Welcome to AAC.
In order to check the PCB layout/tracks we need the PCB images front/back.
E

 

hi DS,
Welcome to AAC.
In order to check the PCB layout/tracks we need the PCB images front/back.
E

These are the PCB traces

 

These are the PCB traces

Here is the top and bottom layers separate, red is top, blue is bottom

 

A couple comments from looking at the schematic.

You need a 100nF cap between Vcc and ground for the CH340.

The LM317 can't/won't regulate the 5 volt Vusb to 5 volts. It must have some overhead to work. This regulator shouldn't be needed.

 

A couple comments from looking at the schematic.

You need a 100nF cap between Vcc and ground for the CH340.

The LM317 can't/won't regulate the 5 volt Vusb to 5 volts. It must have some overhead to work. This regulator shouldn't be needed.

Hello, thanks for the reply, here are the changes made, please let me know if you see anything else that would be wrong, I added an option for 4 pin motors to be attached in the future.

 

Hello, I am finishing up my first PCB and would like someone who has worked with these chips to look over the design I have finished and let me know if they see any issues.

Thanks in advance!

 

ESP32 Modules recommend that you have a board cutout underneath the antenna area. I don't have experience with your CAD system but I don't see any indication of this.

I would recommend giving the datasheet for your part, and it's technical references another look through as they do provide some design guidelines for the pcb layout in regards to the ESP32 modules.