Skip to content

Conversation

@matthewpotter2-nhs
Copy link
Contributor

@matthewpotter2-nhs matthewpotter2-nhs commented Dec 4, 2025

Overview

Jira ticket: NDR-324

Description

Context

Checklist

Tasks for all changes:

  • 1. I have linked this PR to its Jira ticket.
  • 2. I have run git pre-commits. (WIP)
  • 3. I have added and/or updated relevant tests.
  • 4. I have updated relevant documentation.
  • 5. I have considered the cross-team impact (and have PR approval from both Core & Demographics if necessary).
  • 6. I have successfully deployed this change to a sandbox and witnessed unit and e2e tests passing:

Additional tasks for UI changes (delete if not applicable):

  • 1. I have run the UI Smoke Tests against the deployed sandbox and witnessed it passing:
  • 2. I have added evidence (to this PR) e.g. screenshots/gifs of all visual changes.

@matthewpotter2-nhs matthewpotter2-nhs requested review from a team as code owners December 4, 2025 10:49
@adamwhitingnhs adamwhitingnhs changed the title Ndr 324 mapo5 add dev container [NDR-324] Add dev container Dec 4, 2025
jameslinnell
jameslinnell previously approved these changes Dec 5, 2025
jameslinnell
jameslinnell previously approved these changes Dec 8, 2025
jameslinnell
jameslinnell previously approved these changes Dec 9, 2025
Updated the README to include installation instructions for running locally and added pre-requisites for the development environment.
@github-actions
Copy link

Code security issues found

View full details here.

@github-actions
Copy link

Code security issues found

View full details here.

jameslinnell
jameslinnell previously approved these changes Dec 10, 2025
tim-knight-nhs
tim-knight-nhs previously approved these changes Dec 10, 2025
@sonarqubecloud
Copy link

@matthewpotter2-nhs matthewpotter2-nhs merged commit dddbb44 into main Dec 10, 2025
28 checks passed
@matthewpotter2-nhs matthewpotter2-nhs deleted the NDR-324-mapo5-add-dev-container branch December 10, 2025 13:33
lillie-dae pushed a commit that referenced this pull request Dec 11, 2025
Co-authored-by: Kris Bloe <2674722+chrisbloe@users.noreply.github.com>
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.

5 participants