Would like pcb checked if possible

There is a lot I'd change here but lets just go over some basics.

in the schematic

1.  only use perpendicular wires to join components - no odd angles or even 45 degrees, it will make the whole thing more readable.

2.most of your jumpers, JPx are not properly connected in the schematic.

  3. I like to see +ve supply symbols pointing to the top of the board and gnd to the bottom, again this makes a schematic more readable - get some good habits early.

The PCB

1. you've used a ground flood and that's a good thing but one of the most important considerations when laying out a PCB is to consider return current flow and minimise loop areas. This is essential for high speed and good for everything else. You could benefit from a few additional ground connection to tidy up return current loops, or flood  both sides and use appropriate vias, but there is always a compromise because in many cases there are sensitive nodes that should be kept free of ground.  In general read the data sheet and follow any layout guidelines.

2. Decoupling caps should be close to the IC that they are decoupling, e.g U2, C7,C8

3. consider the trace width, this is essential for high current, voltage drop etc, (which you don't really have) and good to keep at a reasonable width for manufacturing, especially if you are going to hand solder ( which I suspect you are).  For hand worked boards I'd not use less than 12 thou.  thicker where possible and for power.

4. you have a lot of components where the orientation should be flipped 180 degrees e.e R4, R16 (180 degress out), this wouel make routing simpler.

5.Always come out of a pad at 0 or 90 degrees never 45 or anything else.

general component placement could be improved considerably, see point 1, and also consider what traces may be high speed and what nodes are low current/sensitive. (not very much here)

It all looks like the autorouter and I've never liked the result produced for single or two layer boards

good luck

There is a lot I'd change here but lets just go over some basics.

in the schematic

1.  only use perpendicular wires to join components - no odd angles or even 45 degrees, it will make the whole thing more readable.

2.most of your jumpers, JPx are not properly connected in the schematic.

  3. I like to see +ve supply symbols pointing to the top of the board and gnd to the bottom, again this makes a schematic more readable - get some good habits early.

The PCB

1. you've used a ground flood and that's a good thing but one of the most important considerations when laying out a PCB is to consider return current flow and minimise loop areas. This is essential for high speed and good for everything else. You could benefit from a few additional ground connection to tidy up return current loops, or flood  both sides and use appropriate vias, but there is always a compromise because in many cases there are sensitive nodes that should be kept free of ground.  In general read the data sheet and follow any layout guidelines.

2. Decoupling caps should be close to the IC that they are decoupling, e.g U2, C7,C8

3. consider the trace width, this is essential for high current, voltage drop etc, (which you don't really have) and good to keep at a reasonable width for manufacturing, especially if you are going to hand solder ( which I suspect you are).  For hand worked boards I'd not use less than 12 thou.  thicker where possible and for power.

4. you have a lot of components where the orientation should be flipped 180 degrees e.e R4, R16 (180 degress out), this wouel make routing simpler.

5.Always come out of a pad at 0 or 90 degrees never 45 or anything else.

general component placement could be improved considerably, see point 1, and also consider what traces may be high speed and what nodes are low current/sensitive. (not very much here)

It all looks like the autorouter and I've never liked the result produced for single or two layer boards

good luck