That seems to be ok in regard to the NRF connections.
Why did you pick 8 instead of the usual 9 for CSN? (I'm just curious)
The 100nF capacitor (probably near the NRF) is quite small, 2 to 10 uF worked quite well for me,
you could also just add a bigger one, keeping the old.
The AMS1117 should have a capacitor on the input and one on the output, near the chip.
Look in the data sheet, I'm quite sure you will find it in the typical application.
These capacitors suppress oscillations, which is a bad thing for a regulator.
And there is the - not commented on - fact, that you have potentiometers on digital only pins.