@Ggreeshma-stani Nailed the PCB in terms of the schematic. But you got it a little bad on the layout side. There are some thumb rules while designing a layout of PCB.
Always place the oscillator and it's capacitors(C1, C2) near to the IC. If you have a doubt on this try checking the Arduino boards or any other boards. you will find an SMD chip crystal alongside of the Arduino XTAL pins. If you ask the reason for this that's simple, to avoid noise creeping in. Crystals provide the clock to the entire digital logic inside the IC. So, there should not be any noise in it. If noise is there, obviously you'll have timing issues.
The bypass caps on the voltage regulator must be placed close to the regulator's input and output pins. The reason for this is also pretty simple, bypass caps will filter out noise coming from the power supply and few other reasons too. So, try placing these bypass caps as close as possible to IC.
Apart from these points, the PCB looked good. Great work. Remember that this message is not to discourage you but just to give a suggestion so that you'll improve it in the next design.