Skip to content

Conversation

@thompson318
Copy link
Contributor

My understanding (from the INFORMus project) is that CSN is stored in the hospital visit table as 'encounter'.

My understanding (from the INFORMus project) is that CSN is stored in the hospital visit table as 'encounter'.
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

PR checklist

This is just a documentation update. So no need for the PR checklist.

skeating
skeating previously approved these changes Dec 6, 2025
Copy link
Contributor

@skeating skeating left a comment

Choose a reason for hiding this comment

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

well spotted

@thompson318 thompson318 requested a review from skeating December 8, 2025 12:49
@thompson318
Copy link
Contributor Author

@skeating I had to add an extra section to emap-tests.yaml in order to be able to merge this PR. If there were no code changes all of the tests were being skipped, so a junit test report was never generated, which is set as a requirement before merging. So the bit I've added will create an empty test report in the event that no tests were required, so we should be able to merge. But now it needs approval again. Please could you take a quick look?

@thompson318 thompson318 merged commit 9200543 into main Dec 11, 2025
13 of 15 checks passed
@thompson318 thompson318 deleted the stephen/csn_clarify branch December 11, 2025 10:36
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.

3 participants