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 69 forks source link

Facility editors should not be able to publish facilities without services #13670

Open swirtSJW opened 1 year ago

swirtSJW commented 1 year ago

User Story or Problem Statement

As a a facility editor I should be prevented from publishing a facility that requires services, when it has no services assigned so that I do not create a bad Veteran experience.

This applies to

Approaches:

Neither of these approaches will remove the risk of a Bulk operation of publish. Bulk operations do not follow entity validation.

Acceptance Criteria

Implementation steps

## Design principles Veteran-centered - [ ] `Single source of truth`: Increase reliability and consistency of content on VA.gov by providing a single source of truth. - [ ] `Accessible, plain language`: Provide guardrails and guidelines to ensure content quality. - [ ] `Purposely structured content`: Ensure Content API can deliver content whose meaning matches its structure. - [x] `Content lifecycle governance`: Produce tools, processes and policies to maintain content quality throughout its lifecycle. Editor-centered - [ ] `Purpose-driven`: Create an opportunity to involve the editor community in VA’s mission and content strategy goals. - [x] `Efficient`: Remove distractions and create clear, straightforward paths to get the job done. - [ ] `Approachable`: Offer friendly guidance over authoritative instruction. - [ ] `Consistent`: Reduce user’s mental load by allowing them to fall back on pattern recognition to complete tasks. - [ ] `Empowering`: Provide clear information to help editors make decisions about their work.
timcosgrove commented 1 year ago

Very much in favor of the first approach:

Entity validation on save: Entity validation is the safest as it will stop it at the time of save and be able to handle a nice error message. It would safely account for any services that were saved in other tabs after the facility form was opened.

I do not think this should be solved via Content Moderation.

mmiddaugh commented 1 year ago

Use cases:

Because of these (and potentially other use cases) we will want to consider whether or not associated services should be a requirement for publishing.