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.

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

MHV-62901: Images request status API implemented #19064

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

mattwrightva
Copy link
Contributor

Summary

  • Images request status API created for checking the status of a radiology images request job

Related issue(s)

https://jira.devops.va.gov/browse/MHV-62901

Research/Implement API call to determine if images have been requested

Description
Research/Implement API call to determine if images have been requested

Lexi and Scott have designs. We have a call that does two things: requests images and give you the request status.

We want to be able to go to the images page without requesting images and also know the status (have we requested them or not?).

Is there an API that will let us know if we have requested them or not. MHV classic does this somehow.

Mike, Muazzam can help/know.

Goal: find the API call to determine if we have requested images. Implement it on vets-api

User Story: As a Medical Images team, we want to research the API call to determine if images have been requested, so that

Testing done

  • Test for the new client method written

Screenshots

no UI changes

What areas of the site does it impact?

MHV medical records

Acceptance criteria

AC1 Research complete and documented
AC2 Call has been implemented into vets-api

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No error nor warning in the console.
  • Events are being sent to the appropriate logging solution
  • Documentation has been updated (link to documentation)
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Feature/bug has a monitor built into Datadog (if applicable)
  • If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
  • I added a screenshot of the developed feature

Copy link
Contributor

@mmoyer-va mmoyer-va left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link

Backend-review-group approval confirmed.

@mattwrightva mattwrightva merged commit 0713aed into master Oct 24, 2024
35 of 36 checks passed
@mattwrightva mattwrightva deleted the MHV-62901-image-request-status branch October 24, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants