department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
99 stars 68 forks source link

Generate a report of what facilities do not have services #7325

Closed kevwalsh closed 6 months ago

kevwalsh commented 2 years ago

User Story or Problem Statement

As a Facility product team member, i want to monitor the quality of facility content.

Implementation steps

Drupal view config to expose service count column.

From July 2023 slack discussion we could add service count column to the main Content audit view, e.g. https://prod.cms.va.gov/admin/content/audit?title_op=contains&title=&type_op=in&type%5B%5D=health_care_local_facility&moderation_state_op=in&moderation_state%5B%5D=editorial-archived&section=All

Acceptance Criteria

Note: The challenge is going to be on the service counts since the counts will be of reverse entity references AND will be to different content types. Doing it as a single column is going to make it such that the column can not be sorted or filtered.... which is not ideal either.

jilladams commented 1 year ago

Tagging @stefaniefgray for awareness of follow ups here re: a view for Facilities with no Services, per https://dsva.slack.com/archives/C02730UEZPS/p1690232280928109?thread_ts=1690228108.808289&cid=C02730UEZPS

jilladams commented 1 year ago

from scrum:

swirtSJW commented 1 year ago

I updated the ACs to match what we need and to better reflect the current state. This is likely not a light lift even though it seems like it should be.

jilladams commented 1 year ago

@swirtSJW what would be required to make it filterable / sortable? Why I ask: the main reason we need this is in order to identify any that should have services but don't. Without the ability to filter / sort, it seems like we are not actually enabling the workflow needed. Or, more accurately: we are unblocking it, but it will still be a pretty toilsome manual process to scroll through and visibly identify 0s, which will be vulnerable to error.

swirtSJW commented 1 year ago

@jilladams The problem is, not all things in a View are filterable and sortable. You can often only filter and sort by things usually that are on the node that shows up in the View. In this case the count is a count of other nodes that are not in any way referenced by the facility node. The connection goes the other way. Services connect to facilities.

There may be some possibilities but it would take some digging.

If we by chance we could get sorting to work, we would not be able to aggregate the columns and would need to have three separate columns (VAMC services, VC Services, and VBA services)

jilladams commented 1 year ago

Discussion in product sync with DaveP, Dotti, Michelle:

jilladams commented 1 year ago

Cross-posting: a simlar need was logged from 7/10 lighthouse sync: https://trello.com/c/1Zr5UVzu/21-facilities-add-a-view-of-the-facilities-with-a-services-count. Not sure why, and if this ticket is about the actual same topic, or if that LH card is something different / for a different purpose. Do we need to document the LH use case for the Service count, and account for it in the work on this ticket?

swirtSJW commented 1 year ago

This should/would cover both needs for HelpDesk and Lighthouse.

swirtSJW commented 1 year ago

Please add your planning poker estimate with Zenhub @omahane

swirtSJW commented 1 year ago

Understood that a single column that is sortable and filterable is ideal. However, we may have to be content with either

  1. Separate columns for VBA counts, VAMC counts, Vet Center Counts that are sortable
  2. CSV export that can be put in a spreadsheet which would allow sorting
jilladams commented 7 months ago

@jilladams double check if this view exists. We think so, and think this can be closed.

jilladams commented 7 months ago

cc @omahane What we have already built is an audit view of VA Services, with a total count, that can be filtered by content type (VAMC, Vet Center health services): https://prod.cms.va.gov/va-services

That satisfies understanding The service, where it's used, and how many times. It does not satisfy understanding which Facilities do not have associated services. Leaving this open.

davidmpickett commented 6 months ago

@omahane I have approved your PR. It passes QA and is doing what it says it is doing.

@jilladams Want to note that the PR does not meet all the ACs listed here. I think we should amend this ticket to just cover the work the PR does.

omahane commented 6 months ago

How about when I do the VBA columns ticket, I try the coalesce? If it turns out to be a bear, we don't do it.

davidmpickett commented 6 months ago

@omahane I stubbed a new ticket for updating the report with VBA and coalesce https://github.com/department-of-veterans-affairs/va.gov-cms/issues/18002

Verified the report is operating as intended on Prod Screenshot 2024-04-30 at 11 17 33 AM