Full name is aria-required in report creation

[chrismytton]

Ensure the full name field is marked as required for screen readers.

Fixes https://github.com/mysociety/societyworks/issues/4492

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.48%. Comparing base (cf7e084) to head (eff944d).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #5169   +/-   ##
=======================================
  Coverage   82.48%   82.48%           
=======================================
  Files         406      406           
  Lines       31728    31728           
  Branches     5048     5048           
=======================================
  Hits        26172    26172           
  Misses       4065     4065           
  Partials     1491     1491           

Flag	Coverage Δ
	82.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dracos]

I think the reason these aren't required is because that can cause issues with the multi-page nature of the form (ie. these being required but on a non-visible page if you're signing in rather than giving your name). The JS in fixmystreet.js already adding required and aria-required and removing, as far as I can see, under "set correct required status depending on what we submit", which @chrismytton you added, so I'd be worried this e.g. made the sign in form unsubmittable for screenreaders. I'd like more details from the person complaining, I think, before we add this, as to the circumstance when it didnt' have aria-required?
Or perhaps aria-required doesn't matter so much. But it's not required if they're signing in is my point, iyswim.
I guess the JS is removing it if you click sign in, so 🤷 guess it doesn't matter if required at the time it loads, hmm.