Skip to content

Conversation

@LilyL0u
Copy link
Contributor

@LilyL0u LilyL0u commented Dec 15, 2025

Resolves JIRA:

Summary

A very high-level summary of easily-reproducible changes that can be understood by non-devs, and why these changes where made.

Code changes

  • A bullet point list of key code changes that have been made.

Developer Checklist

  • UX
    • UX Criteria met (visual UX & screenreader UX)
  • Accessibility
    • Accessibility Acceptance Criteria met
    • Accessibility swarm completed
    • Component Health updated
    • P1 accessibility bugs resolved
    • P2/P3 accessibility bugs planned (if not resolved)
  • Security
    • Security issues addressed
    • Threat Model updated
  • Documentation
    • Docs updated (runbook, READMEs)
  • Testing
    • Feature tested on relevant environments
  • Comms
    • Relevant parties notified of changes

Testing

  • Manual Testing required?
    • Local (Ready-For-Test, Local)
    • Test (Ready-For-Test, Test)
    • Preview (Ready-For-Test, Preview)
    • Live (Ready-For-Test, Live)
  • Manual Testing complete?
    • Local
    • Test
    • Preview
    • Live

Additional Testing Steps

  1. List the steps required to test this PR.

Useful Links

@LilyL0u LilyL0u changed the base branch from latest to WS-197-create-route-in-simorgh-to-render-homepages-nextJS December 15, 2025 20:14
…JS' into migrate-home-page-integration-tests-to-nextjs-app
Base automatically changed from WS-197-create-route-in-simorgh-to-render-homepages-nextJS to latest December 17, 2025 10:07
@LilyL0u LilyL0u marked this pull request as ready for review December 22, 2025 11:32
@amoore108
Copy link
Contributor

This looks great, thanks!

I think some of the other test suites make use of those common utilities that you've ported across. May be worth updating those references for the other page types if there aren't too many? Just so we don't forget to do that later on and it means we can cleanly delete the old folder once all the pages are migrated.

Copy link
Contributor

@MeriemMechri MeriemMechri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@amoore108 amoore108 merged commit 32fc9ed into latest Jan 5, 2026
13 checks passed
@amoore108 amoore108 deleted the migrate-home-page-integration-tests-to-nextjs-app branch January 5, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants