Remove the dedicated MOUs list page#2919
Merged
Merged
Conversation
theseanything
force-pushed
the
remove-mous-list-page
branch
2 times, most recently
from
July 14, 2026 11:17
5d1bcd9 to
d43abd6
Compare
theseanything
force-pushed
the
remove-mous-list-page
branch
3 times, most recently
from
July 24, 2026 11:42
4c9699d to
4a461f4
Compare
The sitemap links to the dedicated MOUs list page, which is being removed now that the organisations pages show each organisation's signed MOUs and agreements. Replace the sitemap's MOUs heading with a link to the organisations page, gated on the organisation policy instead of the MOU policy. The link to the MOU signing page is kept.
The dedicated MOUs list page is being removed now that the organisations pages show each organisation's signed MOUs and agreements. Drop the MOUs item from the header navigation, since super admins can reach the same information through the Organisations item. Update the service navigation component preview to match what super admins now see.
The organisations pages now show each organisation's signed MOUs and agreements, making the standalone list at /mous redundant. Remove the route, the controller's index action and its authorisation callbacks, the view and its spec, and the locale strings only that view used. The MOU signing flow is unchanged.
Its last caller was the MOUs list page's index action, which has been removed. The sitemap and header navigation now use the organisation policy instead.
The mou_signatures.index locale namespace referred to the MOUs list view, which no longer exists. Its surviving keys are only used by the organisations pages, so home them where they are consumed: the agreement type labels move to mou_signatures.agreement_type as they describe the model's enum and are shared between the organisations index, show and filter, while the table headings, preamble and share links move under organisations.show.mou_signatures alongside the existing heading. This stops the keys looking like dead code tied to a deleted view.
theseanything
force-pushed
the
remove-mous-list-page
branch
from
July 24, 2026 11:58
4a461f4 to
11f1594
Compare
theseanything
marked this pull request as ready for review
July 24, 2026 11:59
|
🎉 A review copy of this PR has been deployed! You can reach it at: https://pr-2919.admin.review.forms.service.gov.uk/ It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready For the sign in details and more information, see the review apps wiki page. |
stephencdaly
approved these changes
Jul 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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 problem does this pull request solve?
Removes the standalone MOUs list page at
/mousin favour of the MOUs and agreements shown on the organisation pages.The organisations pages list each organisation's signed MOUs and agreements (and the organisations index shows MOU-signed status with filtering), so the separate list page is redundant. The header navigation and sitemap now point at the organisations pages instead, and the
MouSignaturePolicyis removed as nothing uses it any more. The MOU signing flow is unchanged.The locale keys the list page shared with the organisation pages have also moved out of the
mou_signatures.indexnamespace, since there is no longer an index page for them to belong to.