"...on the led both ratlines go to one end of it." No they don't:  There are several things wrong with your schematic. 1. It is incomplete; 2. It is not updated to the current schematic (the LED is called LED and not LED1);  3. The component placement around the SMPS chip U1 does not conform to the recommendations given in the device datasheet: [https://www.analog.com/media/en/technical-documentation/data-sheets/1370fs.pdf](https://www.analog.com/media/en/technical-documentation/data-sheets/1370fs.pdf) and in the Application Note 19 (AN19): [https://www.analog.com/media/en/technical-documentation/application-notes/an19fc.pdf](https://www.analog.com/media/en/technical-documentation/application-notes/an19fc.pdf); 4. LED1 cathode is connected to the output of a positive output voltage linear regulator, U2. From the rest of the incomplete schematic, it would appear that you are intending to connect the currently unconnected end of LED1 (presumably through a resistor) to ground. If you want the LED to illuminate when the supply out of U2 is on then you must connect the anode of LED1 to the OUT pin of U2 and the cathode end to ground; 5. There is no current limiting resistor in series with the LED;  6. The Supplier selected for D1 is Digikey but all other BoM attributes are for LCSC;  7. The clearances around the USB connector are less than the clearances you have set in the Design Rules: 

Thank you! I really appreciate you taking the time to look at it, when I was working on the schematic yesterday there was a wire connecting everything to ground but when I logged on today the wire wasn't there, I assume it was a connectivity error. I added the wire and the led resistor you mentioned as well as fixing the clearance, will using 5mil give me problems? As for the U1 components I assume you're referring to the resistor values, and the ones I use is copied from resistor values someone else used in their project and as I far as I understand it will just change the output Voltage from the intended 12v to 30v. As for the supplier, is that an issue? I already have the part from digikey and changed the size of the holes as the digikey diode i'm using has a lower gauge wire. This is just for my own personal use and not a commercial project if that was the issue.

@andyfierman forgot to @ you

# "...fixing the clearance, will using 5mil give me problems?" The clearance you need to use is defined by the pad spacing of the USB connector but the lower limit is defined by the JLCPCB process capabilities: [https://jlcpcb.com/capabilities/Capabilities](https://jlcpcb.com/capabilities/Capabilities) # "...As for the U1 components I assume you're referring to the resistor values,..." Sorry: I should have said "There are several things wrong with your schematic and PCB.". In my point (3) I am referring to the physical placement of the parts in the PCB. I cannot comment on your actual values since you have not published any kind of **Design Requirements Specification** or **How it works** document and have not annotated the schematic to indicate input or output voltages. Please see (2.2) in: [https://easyeda\.com/andyfierman/Welcome\_to\_EasyEDA\-31e1288f882e49e582699b8eb7fe9b1f](https://easyeda.com/andyfierman/Welcome_to_EasyEDA-31e1288f882e49e582699b8eb7fe9b1f) # "As for the supplier, is that an issue?" For you maybe not but: 1. it is good practice for you to have a properly documented design and; 2. confusing for others who may want to copy your design.