Skip to content

Conversation

@amoore108
Copy link
Contributor

Summary

  • Resolves issue with parsing the service from the path in _app
  • This was highlighted with the release of Homepages on Next.js, as the likes of /kyrgyz.lite resulted in a service of kyrgyz.lite instead of just kyrgyz

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

@amoore108 amoore108 self-assigned this Jan 6, 2026
@amoore108 amoore108 merged commit 9a61a8c into latest Jan 6, 2026
13 checks passed
@amoore108 amoore108 deleted the NOTICKET-fix-service-parsing-in-app branch January 6, 2026 12:25
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