Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this pull request do?
This PR does two things:
I targeted only handling 404s differently as I don't know enough yet about other errors and the data structure that might come from LAALAA. Error handling is done quite weirdly, and the form is weirdly linked into the data response so haven't touched that. Most sensible place to raise 404 felt like the view given the current code structure.
Previously, 404s were being passed down to the forms/view weirdly, which meant the view was receiving no data structure - so everything blew up at the point of pagination. Have made the movement of data more sensible and now catching the 'Invalid page' with the 404.
Any other changes that would benefit highlighting?
Needs some good UAT to make sure I haven't inadvertently broken how other errors are returned and to make sure pagination works as expected.
Checklist