No obvious issues with what you chose. I would recommend that you partition the disk into C: and D:. Make the C: portion under 240GB. That way when you have some more money you can easily move the C: partition to an SSD.
If you include a link to your pcpartpicker list, we can see what choices you did make rather than starting from scratch.
Thanks for your help but I have change my build and it is affordable now if you would like to see my new build here it is https://au.pcpartpicker.com/list/zsVLRG
No obvious issues with what you chose. I would recommend that you partition the disk into C: and D:. Make the C: portion under 240GB. That way when you have some more money you can easily move the C: partition to an SSD.