I don't see any problems with that.
However, I'd like to make some observation. If your whole design is mostly surface mount, it would be sort of a shame to use an electrolytic capacitor which would require you to have holes in the circuit board just for that capacitor. Of course there are surface mount electrolytic capacitors but what I found is that they're less reliable than capacitors with leads, they're more expensive, they take up more space and they're also quite tall (an electrolytic capacitor with leads you could in theory lay flat on the motherboard like a crystal oscillator or a plain resistor)
There are polymer capacitors but they're quite expensive, a few times more expensive compared to electrolytic... for example this one is 1$ if you buy 25, or 0.5$ if you buy 1000 :
https://www.digikey.com/product-detail/en/panasonic-electronic-components/EEF-CD1C4R7R/PCE3170CT-ND/256710If the regulator says it's stable with ceramic capacitors, maybe just using a second 2.2uF or 3.3uF or 4.7uF x5r or x7r and with decent voltage rating (let's say 25v or higher) would be enough, for example this 4.7uF 25v x5r is 5.5 cents each if you buy 100 or 16 cents if you buy just one :
https://www.digikey.com/product-detail/en/murata-electronics-north-america/GRM21BR61E475MA12L/490-5422-1-ND/2175229Some other notes.
It will probably make your board look nicer and also easier to route if you use 45 degree angles in your traces - i see you seem to use only right angle (90 degrees).
It's not wrong and doesn't make much difference, but I generally like to see components like resistors (that 820 ohm resistor and R45 in your case) close to the end of a trace, in your case closer to the headers unless their height would physically block insertion of something in the header, or there could be some risk of mechanical pressure on the resistors.
Also, I'd say in general you should try to aim for as few vias as possible (jumping traces from one layer to another). For simple designs, ideally you'd want to aim for keeping it all on one layer. You have a few places where you jump on the other layer almost just for the fun of it, basically it can be avoided with a bit more clever routing.
For example, the two traces on top right of the header could be moved all the way to the edge of the board, and go down along the edge of the board this way making the switch easier to connect (and no need to go near the switch to the other layer (btw there's some visual bug there, you go at the switch on bottom layer but don't see the via coming out to connect to yellow trace)
This will also save you going on other layer at the 820 ohm resistor since now you'd have space to put the resistor right there near the header.
Another less common technique to reduce the number of vias is to use 0 ohm resistors as a convenient way to jump over traces. For example, you have those 3 traces going on the other layer just to cross three other traces.
There are surface mount resistors which you could use to jump over the traces, depending on their length and the trace thickness it would be easy to jump over 2-3 traces.
It's less common because it adds some cost (vias are free but resistors are fractions of a penny) and often it's not used because it increases the completion time for pick and place machines or the pick and place machines have a limited number of parts they can work with (for example a machine could only pick parts from 64 tapes of components, so adding your 0 ohm resistor would block a tape that could be used for other chips)
However, it's a design with a small number of units made, it could make the layout look nicer and leave you the bottom space for routing traces or could even allow you to use a cheaper single layer pcb
Here's some examples of such resistors, 0805 (probably wide enough for 2 traces to go under it) 1 cents each if you buy 50 :
https://www.digikey.com/product-detail/en/yageo/RC0805JR-070RL/311-0.0ARCT-ND/7311631206 is about the same price :
https://www.digikey.com/product-detail/en/yageo/RC1206JR-070RL/311-0.0ERCT-ND/732131