Skip to content

Conversation

@eesa456
Copy link
Contributor

@eesa456 eesa456 commented Dec 13, 2024

…er and error message fix

@eesa456 eesa456 changed the title NRL-1076 add integration and unit tests for identifier accession numb… NRL-1076 add integration and unit tests for accession number Dec 13, 2024
@github-actions
Copy link

💥 Something went wrong while building the pull request environment.
Check Output Logs

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 4abb3e3724ccce686c68812c02021eb52615f301
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

When consumer 'RX898' searches for DocumentReferences with parameters:
| parameter | value |
| subject | 9278693472 |
| type | http://snomed.info/sct\|887701000000100 |
Copy link
Contributor

Choose a reason for hiding this comment

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

This might not be doing what you expect it to do. See:

if type_code := items.get("type"):

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm searching for pointers with that type as a parameter thats got nothing to do with creating a pointer of a certain type?

Copy link
Contributor

Choose a reason for hiding this comment

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

True, it's a search, ignore me

When consumer 'RX898' searches for DocumentReferences using POST with request body:
| key | value |
| subject | 9278693472 |
| type | http://snomed.info/sct\|887701000000100 |
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, see:

if type_code := items.get("type"):

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah its a parameter for the search request not creating a pointer

Copy link
Contributor

Choose a reason for hiding this comment

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

True, it's a search, ignore me

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 1754771ae617e936e67824a6bd239b4487ce4e9b
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 1754771ae617e936e67824a6bd239b4487ce4e9b
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 123f8379b53a5c5f1384e92dd60164f470df3cff
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

When consumer 'RX898' searches for DocumentReferences with parameters:
| parameter | value |
| subject | 9278693472 |
| type | http://snomed.info/sct\|887701000000100 |
Copy link
Contributor

Choose a reason for hiding this comment

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

True, it's a search, ignore me

When consumer 'RX898' searches for DocumentReferences using POST with request body:
| key | value |
| subject | 9278693472 |
| type | http://snomed.info/sct\|887701000000100 |
Copy link
Contributor

Choose a reason for hiding this comment

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

True, it's a search, ignore me

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 123f8379b53a5c5f1384e92dd60164f470df3cff
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 9c16a6c97e11f62b35921d0d3be83917c5dba166
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

@sonarqubecloud
Copy link

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 19ad61573a18c25f42fda27a07cdf8cba432c0c8
URL: https://nrl1076-e84289.api.record-locator.dev.national.nhs.uk/

@eesa456 eesa456 merged commit 32d461f into develop Dec 13, 2024
8 checks passed
@eesa456 eesa456 deleted the feature/eema1-NRL-1076-IdentifyLinkedStudiesAndReports branch December 13, 2024 15:51
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.

2 participants