Your only ground connection to the NodeMCU board is through the button which is not the best idea. You should have at least one trace connecting it directly. Even if it’s just putting a trace between the pins on the button.
Is there a reason why you have a 10k pullup on the humidity sensor that you did not have on the last version?
Now assuming that the NodeMCU part you used is labelled correctly I think you have everything connected to the correct locations.
With all of that said I would not use that NodeMCU part because of how small the rings are around the pins. I would also not route any of the traces between the pins either. If you move the NodeMCU board to the left and move all the parts from the left to the right you will have only 4 traces to route across the PCB instead of 15 which means you will have plenty of room to route them around the ends.
The last thing is GPIO16 (NodeMCU D0) is a special pin and may cause board resets if you try and use it.