Seems okay.
May want more vias around the MCU. Can't quite tell which pins are ground, if any, but those should have vias near them. Same for the GPS module; it has some nearby (including flanking the antenna trace, that's nice), but I don't know at a glance what connects where.
Don't know what the datasheet says, but the GPS module might want traces out from under it, you'll want to look that up. You have tons of routing area to run traces around the outside, and stitch ground around the traces beside, so that's no problem.
What does the purple outline imply? Is that a courtyard that nearby components should be outside of, or something? I'm guessing not, as capacitors are within it.
Huh... is there no input bypass cap? Hard to tell without schematic, but it looks like that is the case. The charger and regulator likely both would appreciate having such. Beware of mechanical switching between large reservoirs (i.e. Li cell, ceramic capacitor), as inrush and voltage overshoot can occur; some series resistance (for a low-power device like this, even just an ohm in series with the switch might be adequate -- this does depend on the 915MHz radio's transmit power, mind), or a TVS at the regulator input, may be desirable.
Tim