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

Merge Mental Health phone number integration branch to main #15576

Closed
2 of 5 tasks
jilladams opened this issue Oct 5, 2023 · 5 comments
Closed
2 of 5 tasks

Merge Mental Health phone number integration branch to main #15576

jilladams opened this issue Oct 5, 2023 · 5 comments
Assignees
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) sitewide

Comments

@jilladams
Copy link
Contributor

jilladams commented Oct 5, 2023

BLOCKED - until Lighthouse is ready to receive the Mental Health data

description

All of the of the Mental health phone number CMS work in this epic will be PRs against the integration branch integration-mental-health-phone since it can not go directly to main.

Notes

We don't currently have a Lighthouse data push-specific ticket, so that work should be included in the estimate here.
(LH trello card

Acceptance criteria

  • Lighthouse / Drupal eng have verified that Mental health numbers are pushed / received correctly
  • Integration branch PR is merged to main
  • Features are verified live in production
  • I expect the content model documentation is updated to show that the field_mental_health_phone is both a push and a pull
  • I expect the content model documentation is updated to show that the field_mental_health_contact_phon is a push, and is short lived and expected to be removed in the final phase of this.
@jilladams jilladams added Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) labels Oct 5, 2023
@swirtSJW swirtSJW changed the title Merge integration branch to main Merge Mental Health phone number integration branch to main Oct 6, 2023
@jilladams jilladams added Blocked Issues that are blocked on factors other than blocking issues. Needs refining Issue status labels Oct 23, 2023
@xiongjaneg
Copy link
Contributor

per 10/23 meeting, potentially early December for LH timeline

@mmiddaugh
Copy link
Contributor

mmiddaugh commented Oct 24, 2023

Per email from VHA DM, we will wait to merge until LH is ready
fyi @xiongjaneg @jilladams @swirtSJW

should we move this to blocked?

@jilladams
Copy link
Contributor Author

jilladams commented Jan 16, 2024

Per Lighthouse, they'll be shipping their change to be ready for this today, Tues Jan 16, 2024.

https://dsva.slack.com/archives/CJ162GDDJ/p1704986030785639?thread_ts=1704982321.370329&cid=CJ162GDDJ

@jilladams jilladams removed the Blocked Issues that are blocked on factors other than blocking issues. label Jan 16, 2024
@swirtSJW
Copy link
Contributor

This has been merged. I am blocked from completing the last three AC's.

  1. the code has not made it to prod yet
  2. I can not update the content model documentation because those are currently un-editable due to a alt text validation bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) sitewide
Projects
None yet
Development

No branches or pull requests

4 participants