Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Meds and Allergies incorrect responses related to clinically asserted no known allergies vs no allergies recorded. #365

Closed
SimonFarrowNHS opened this issue Feb 10, 2021 · 4 comments
Closed

Meds and Allergies incorrect responses related to clinically asserted no known allergies vs no allergies recorded. #365

SimonFarrowNHS opened this issue Feb 10, 2021 · 4 comments
Assignees
@SimonFarrowNHS
Labels
1.5.0 bug Structured

Comments

@SimonFarrowNHS
Copy link
Contributor

Email dated 18/1/2021 14:27
Test cases referenced here:
GPC-STR-TST-ALG-08 Using the allergies resource (Clinically asserted no known allergies) Test patient: 9658218989
GPC-STR-TST-ALG-07 Using the allergies resource (No data response) Test patient: 9658218865

Supplier has reported incorrect behaviour in this area. BAs agree. Further investigation revealed an ambiguity in the spec covering this area. Further work to fix this is currently suspended pending publication of the amended specification as at 10/2/2021.
@SimonFarrowNHS SimonFarrowNHS self-assigned this Feb 10, 2021
@SimonFarrowNHS
Copy link
Contributor Author

SimonFarrowNHS commented Feb 17, 2021 •
edited
Loading

In the gp spec repo nhsconnect/gpconnect#1038 refers.

Email trail on 13/1/21 also refers.

@SimonFarrowNHS
Copy link
Contributor Author

Further investigation and correspondence shows that the Active allergies display should be the same as the list title.
ie Allergies and adverse reactions

SimonFarrowNHS added a commit that referenced this issue Mar 12, 2021
@SimonFarrowNHS
Addresses #365 null flavours of no allergies
cb76475
@SimonFarrowNHS
Copy link
Contributor Author

Question is do we apply this change to 1.2 as well?

SimonFarrowNHS added a commit that referenced this issue Mar 26, 2021
@SimonFarrowNHS
Changes for #365
ddecacb
@SimonFarrowNHS
Copy link
Contributor Author

Change now also applied to release/1.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SimonFarrowNHS SimonFarrowNHS

Labels
1.5.0 bug Structured
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SimonFarrowNHS