Yes, that looks fine - as @maze1980 says the charger may well not provide any power without a battery also attached.
Only thing I'd say is there's actually a specific set of 2mm spaced holes for the JST connector (see the pic) - these have the same connections as you used but it allows you to still add pins, or to use the power pins down the side to connect other stuff.
Espruino is a JavaScript interpreter for low-power Microcontrollers. This site is both a support community for Espruino and a place to share what you are working on.
Yes, that looks fine - as @maze1980 says the charger may well not provide any power without a battery also attached.
Only thing I'd say is there's actually a specific set of 2mm spaced holes for the JST connector (see the pic) - these have the same connections as you used but it allows you to still add pins, or to use the power pins down the side to connect other stuff.
1 Attachment