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

SPIKE: Do we need a code exception for Manila and the Get Updates box? #19272

Closed Agile6MSkinner closed 1 month ago

Agile6MSkinner commented 1 month ago

There's a Get Updates box at the bottom of the facility pages. Is there something about Manila that would require exceptions to be made in the code?

Background

Agile6MSkinner commented 1 month ago

Hey team! Please add your planning poker estimate with Zenhub @dsasser @eselkin

eselkin commented 1 month ago

Probably needs a conditional to handle it.

eselkin commented 1 month ago

When I publish Manila VA Clinic or Manila VA System I get the Get Updates Box, so not sure why we're saying it does't work. Maybe if only one of those is published it doesn't work? Like without the system or something? But I'd need @davidmpickett to identify which gets published the system or the clinic only or do both get published, but since the clinic doesn't link to the system then it just shows the clinic as the base page?

davidmpickett commented 1 month ago

We don't want the Manila VA System to publish a page on the front end. So when I was doing my proof of concept, I left the System unpublished in Drupal. I was thinking we would want to keep the System unpublished, but maybe instead it's how to prevent it from building a page in Content Build even when published in Drupal

eselkin commented 1 month ago

With just Manila VA Clinic published (not the VA System). I get the Updates block and the correct GovDelivery codes ("1") for the signups as in the CMS. However, I do not get the side nav, but I think that's another ticket. However, it took about 2.5 hours for me to build from my tugboat and my tugboat completely would not build on the tugboat itself.

However, the VA System has to publish and then archive afterward for changes to update on the system for them to show up on the Clinic.

eselkin commented 1 month ago

So the limitation on side nav is due to: https://github.com/department-of-veterans-affairs/vets-website/blob/06563d07360e3960525c69dd60e73e8a7bfcd8a7/src/platform/site-wide/side-nav/helpers.js#L48

I'll put the rationale in the other ticket. It's coming across correctly from content-build to vets-webiste. But v-w processes out the dependent pages for some reason.

eselkin commented 1 month ago

So if you change the system updates/links section you get the correct links in the Clinic page: Image This seems completely fine and acceptable, right?

RE: Menu items However, if you add the Performance menu item to the clinic (not the system), we see this type of issue: Image

This is because it's not in a level below the header?

jilladams commented 1 month ago

Follow ups

eselkin commented 1 month ago

Archive -> Draft does update the system Get Updates information Image

jilladams commented 1 month ago

Nice! I think that means the 2 things left to sort are:

  1. Drupal: a mechanism to never let Manila System move to Published.
    • In prod, we need to move to Published > Archived, nowish.
    • After that move, you can go Archive > Draft, and Draft > Archive, and Manila System should never be allowed to publish again.
  2. The menu questions.
jilladams commented 1 month ago

From Slack : https://dsva.slack.com/archives/C0FQSS30V/p1727974260585079 Plan for managing the system:

Right now: we publish > Archive the system node, so it's handled for launch. We make a ticket to prevent any future publishing of the System to the FE, a la Lovell. Not a launch blocker, immediate fast follow

I stubbed this ticket for fast follow: https://github.com/department-of-veterans-affairs/va.gov-cms/issues/19406

jilladams commented 1 month ago

Today I published / Archived the prod VAMC System, FYI.

@eselkin is the rest of the menu stuff here resolved? I feel like we have tickets in sprint knocking out the actual wokr, and we can say this is done?

eselkin commented 1 month ago

Right. For the system that's it.