Skip to content

WS-NA-10-time: Adds live pulse to nav [copilot]#13621

Draft
Isabella-Mitchell wants to merge 3 commits intolatestfrom
WS-NA-10-time-add-live-pulse-nav
Draft

WS-NA-10-time: Adds live pulse to nav [copilot]#13621
Isabella-Mitchell wants to merge 3 commits intolatestfrom
WS-NA-10-time-add-live-pulse-nav

Conversation

@Isabella-Mitchell
Copy link
Contributor

@Isabella-Mitchell Isabella-Mitchell commented Jan 16, 2026

Resolves JIRA: N/A (10% time)

Summary

Adds a Live Pulse to navigation for Arabic Live Broadcast page link. (A training exercise to use co-pilot agent & prompting)

Screenshot 2026-01-16 at 14 55 12 Screenshot 2026-01-16 at 14 55 18 Screenshot 2026-01-16 at 14 55 26

Code changes

  • Hardcodes in service config
  • Looks for this value in Nav, if it exists then adds a live pulse.

To do

  • Understand if page is ever not "live". If so, see if can we remove pulse for this
  • Add Toggle if desired
  • A11y & UX consultation
  • Add story/ testing
  • Remove on lite?
  • Add further services

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

Launch NextJs server locally
Visit http://localhost:7081/arabic?renderer_env=live

Useful Links

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.

1 participant

Comments