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

VACMS-15901: Removes facilities not in user sections #16663

Merged
merged 2 commits into from
Jan 9, 2024

Conversation

omahane
Copy link
Contributor

@omahane omahane commented Jan 4, 2024

Description

Relates to #15901
Relates to #15391
Relates to #7538
Relates to #12737
Relates to #15906

Testing done

Manually
Cypress

Screenshots

VBA Facility Service (intended content type)

Screenshot 2024-01-08 at 1 12 43 PM

Events (an affected content type)

Screenshot 2024-01-05 at 3 34 57 PM

QA steps

What needs to be checked to prove this works?

  • Create facility service as editor in Cheyenne VA Regional Benefit Office
    • Log in as "test_Chanelle18"
    • Create a VBA Facility service
    • Confirm that the Facility dropdown only shows facilities in the "Cheyenne VA Regional Benefit Office" section
    • Choose "VA Regional Benefit Satellite Office at Sheridan VA Medical Center" from the Facility dropdown
    • Choose "Home Loans" from the Service name dropdown
    • Save the node
    • Log out
  • Edit facility as editor in Cheyenne and Columbia sections
    • Log in as "test_Lucy.Abernathy30"
    • Edit the "Home loans" service you just created
    • Confirm that the Facility dropdown only shows facilities in the Cheyenne and Columbia sections

What needs to be checked to prove it didn't break any related things?

  • Spot-check one content type for each of the winnowed fields
    • Create an Event (to check field_listing)
      • Log in as "QA Content Publisher," an editor in only one section
        • Create an event
        • Confirm that you can only choose a facility in the user's section from the Where should the event be listed? dropdown.
      • Log in as "Victor.A.MCtest," an editor in the "Outreach hub" and other sections
      • Create an event
        • Confirm that you can only choose a facility in the user's sections or "Outreach and events: outreach events" from the Where should the event be listed? dropdown.
    • Create a VAMC Facility Health Service (to check field_facility_location)
      • As Victor.A.MCtest, create a VAMC Facility health service
      • Confirm that you can only choose a facility in one of the user's sections from the Facility dropdown.
    • Create a VAMC Detail Page (to check field_office)
      • As Victor.A.MCtest, create a VAMC Detail Page.
      • Confirm that you can only choose a VAMC system in one of the user's sections from the Related office or health care system dropdown.

Testing Google spreadsheet:
https://docs.google.com/spreadsheets/d/17WXvo_QAPvB02Xlm0_np1uQWy83u3dbfJnRxAQdG_h0/edit#gid=0

Definition of Done

  • Documentation has been updated, if applicable.
  • Tests have been added if necessary.
  • Automated tests have passed.
  • Code Quality Tests have passed.
  • Acceptance Criteria in related issue are met.
  • Manual Code Review Approved.
  • If there are field changes, front end output has been thoroughly checked.

Select Team for PR review

  • CMS Team
  • Public websites
  • Facilities
  • User support
  • Accelerated Publishing

@omahane omahane changed the title VACMS-15901: Removes facilities not in user's sections VACMS-15901: Removes facilities not in user sections Jan 5, 2024
@JunTaoLuo JunTaoLuo closed this Jan 5, 2024
@JunTaoLuo JunTaoLuo reopened this Jan 5, 2024
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 5, 2024 14:40 Destroyed
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 5, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 5, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 6, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 7, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 8, 2024
@omahane omahane requested a review from swirtSJW January 8, 2024 21:53
@github-actions github-actions bot added the Facilities Facilities products (VAMC, Vet Center, etc) label Jan 8, 2024
@omahane omahane force-pushed the VACMS-15901-winnow-VBA-facility-dropdown branch from 31978e3 to 81aa703 Compare January 8, 2024 22:15
@va-cms-bot va-cms-bot temporarily deployed to Tugboat January 8, 2024 22:15 Destroyed
@omahane omahane requested a review from jilladams January 8, 2024 22:28
@jilladams
Copy link
Contributor

jilladams commented Jan 8, 2024

Tested as 3 different editors who have multiple section permissions: 2 that have 1 VAMC + Outreach Hub, and 1 that has 2 VAMC sections. In all 3 cases, the Section options match the Event listing options, and Event saved correctly. Just requested a content release for kicks, but the FE shouldn't have anything to do with the change here. So: LGTM.

The events I made:

https://pr16663-qymzgx6nhcrlbdd8ahjuxg5xqw6l4sff.ci.cms.va.gov/northeast-ohio-health-care/events/64718
Screenshot 2024-01-08 at 2 56 59 PM

  • VA NE Ohio & Outreach Hub Sections
  • Published to Outreach Hub also

https://pr16663-qymzgx6nhcrlbdd8ahjuxg5xqw6l4sff.ci.cms.va.gov/northern-arizona-health-care/events/64719
Screenshot 2024-01-08 at 3 08 10 PM

  • VA Northern Arizona & Outreach Hub Sections

https://pr16663-qymzgx6nhcrlbdd8ahjuxg5xqw6l4sff.ci.cms.va.gov/bedford-health-care/events/64720
Screenshot 2024-01-08 at 3 19 29 PM

  • VA Bedford & VA Boston Sections

@jilladams
Copy link
Contributor

Added "Relates to" in ticket body to all the tickets we think we might be solving here. Each'll need to be verified by the relevant team, but this'll help with paper trail.

@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
@JunTaoLuo JunTaoLuo added the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
@github-actions github-actions bot removed the refresh-tugboat-cache Used by GHA Tugboat automation label Jan 9, 2024
Copy link
Contributor

@swirtSJW swirtSJW left a comment

Choose a reason for hiding this comment

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

This tests out flawlessly and code is precise. Great work @omahane
image

@swirtSJW
Copy link
Contributor

swirtSJW commented Jan 9, 2024

image

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

Successfully merging this pull request may close these issues.

5 participants