-
Notifications
You must be signed in to change notification settings - Fork 58
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
Appointment search using VPG #16794
Appointment search using VPG #16794
Conversation
Generated by 🚫 Danger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good, but it appears that the size of the PR has tripped the LOC limit. I wonder if some of the tests could be added to a separate ticket.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mobile changes look good to me. Are there any functional differences between VAOS and VPG?
No, for now VPG is using VAOS as the source |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for VAOS
This reverts commit c40e3a1.
Summary
va_online_scheduling_enable_OH_reads
feature flagvpg
endpoint instead ofvaos
Related issue(s)
https://app.zenhub.com/workspaces/appointments-oracle-health-integration-65a6e99ea522640e4d09393b/issues/gh/department-of-veterans-affairs/va.gov-team/83349
Testing done
What areas of the site does it impact?
These changes impact health appointment creation.
Acceptance criteria