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

[Blocked] FE: Top pages menu: Make H2 dynamic /read from CMS menu title #11568

Closed
1 task
jilladams opened this issue Nov 14, 2022 · 3 comments
Closed
1 task
Labels
Blocked Issues that are blocked on factors other than blocking issues. Defect Something isn't working (issue type) Needs refining Issue status Public Websites Scrum team in the Sitewide crew sitewide VA.gov frontend CMS team practice area

Comments

@jilladams
Copy link
Contributor

jilladams commented Nov 14, 2022

BLOCKED: the editable field in the CMS gets overwritten from code, so you cannot actually modify the menu name via editable CMS field

Menu title

#11654 will update the H2 name of this menu from Popular on VA.gov > Top pages

Section title is currently hard coded: https://github.com/department-of-veterans-affairs/content-build/blob/main/src/site/includes/common-tasks.drupal.liquid#L48

The menu title can be edited in the CMS, and the change doesn't affect the machine name: https://prod.cms.va.gov/admin/structure/menu/manage/popular-on-va-gov

Dynamically populate this H2. Likely only Drupal Admins can save a menu name change, but dynamic H2 would need handled for line-wrapping / worst case line length, etc.

Acceptance Criteria

@jilladams jilladams added Defect Something isn't working (issue type) VA.gov frontend CMS team practice area Public Websites Scrum team in the Sitewide crew labels Nov 14, 2022
@jilladams jilladams mentioned this issue Nov 15, 2022
34 tasks
@jilladams jilladams changed the title FE: order of menu links should mirror CMS order FE: order of menu links should mirror CMS order & enabled / disabled status Nov 17, 2022
@jilladams jilladams added the Needs refining Issue status label Nov 18, 2022
@jilladams jilladams changed the title FE: order of menu links should mirror CMS order & enabled / disabled status FE: Popular on VA.gov menu fixes for order, showing enabled links only, section title Nov 18, 2022
@jilladams
Copy link
Contributor Author

Removing from scope:

Enabled vs. Disabled links

The front-end should only show enabled menu links:

Screen Shot 2022-11-17 at 3 43 50 PM

This may require help from Drupal folk for updated graphQL to account for this?

Menu Order

For several days, the Popular on VA.gov menu showed the last 2 links out of order, vs. what's in the Prod CMS.
Expected behavior: Order of Popular on VA.gov (and Homepage Hub List) menu links in the front-end should mirror the CMS order.

This no longer repros, but we want to confirm that in the code, order is set within the CMS and respected by the front-end.

What was happening:

Screen Shot 2022-11-14 at 12 13 54 PM

why

We verified that:

  1. Disabled links: GraphQL doesn't send through disabled links to front end. The diff of why a disabled link continued to show, is likely due to lag from Prod data change > nightly CMS build > content releaase > mirror to Staging.
  2. Order: same.

@jilladams jilladams changed the title FE: Popular on VA.gov menu fixes for order, showing enabled links only, section title FE: Top pages menu: Make H2 dynamic /read from CMS menu title Nov 18, 2022
@jilladams jilladams added the Blocked Issues that are blocked on factors other than blocking issues. label Dec 6, 2022
@jilladams
Copy link
Contributor Author

To unblock this: we need to confirm priority with Dave.

If it's a priority:

  • CMS needs some thinking & a ticket to make the editable fields persist. Right now they are overwritten by config on every build.
  • Then: update this ticket with any notes that are relevant to front-end.

@jilladams jilladams changed the title FE: Top pages menu: Make H2 dynamic /read from CMS menu title [Blocked] FE: Top pages menu: Make H2 dynamic /read from CMS menu title Dec 27, 2022
@FranECross
Copy link

Closing old iceboxed ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Issues that are blocked on factors other than blocking issues. Defect Something isn't working (issue type) Needs refining Issue status Public Websites Scrum team in the Sitewide crew sitewide VA.gov frontend CMS team practice area
Projects
None yet
Development

No branches or pull requests

2 participants