Skip to content

VED-1054: Add API documentation for search by disease type#1276

Open
Thomas-Boyle wants to merge 4 commits intomasterfrom
1054-target-disease-api-docs
Open

VED-1054: Add API documentation for search by disease type#1276
Thomas-Boyle wants to merge 4 commits intomasterfrom
1054-target-disease-api-docs

Conversation

@Thomas-Boyle
Copy link
Contributor

  • Updated the API specification to clarify the use of the new target-disease
    parameter alongside -immunization.target for searching vaccination records.
  • Improved descriptions for both parameters to ensure clear guidelines on their usage and mutual exclusivity.
  • Added examples and links to supported target disease codes to aid developers in implementing the search feature effectively.
  • Ensured consistency in error handling documentation for invalid parameter combinations.

…e search functionality

- Updated the API specification to clarify the use of the new `target-disease` parameter alongside `-immunization.target` for searching vaccination records.
- Improved descriptions for both parameters to ensure clear guidelines on their usage and mutual exclusivity.
- Added examples and links to supported target disease codes to aid developers in implementing the search feature effectively.
- Ensured consistency in error handling documentation for invalid parameter combinations.
Copy link
Contributor

@edhall-nhs edhall-nhs left a comment

Choose a reason for hiding this comment

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

Nice clear & helpful information 👍 Only suggested a few formatting changes for consistency

dlzhry2nhs
dlzhry2nhs previously approved these changes Mar 6, 2026
JamesW1-NHS
JamesW1-NHS previously approved these changes Mar 6, 2026
- Updated descriptions for the `target-disease` parameter to clarify its expression format as `system|code`.
- Ensured consistency in language throughout the documentation to enhance clarity for developers implementing the search functionality.
- Maintained links to the supported target disease list for reference.
@Thomas-Boyle Thomas-Boyle dismissed stale reviews from JamesW1-NHS and dlzhry2nhs via 5492a80 March 6, 2026 15:07
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox March 9, 2026 08:39 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox March 9, 2026 08:40 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

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.

4 participants