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

[Social Media Links - Benefit Hub pages] Social media links in the Follow us section of these pages are broken #18265

Closed FranECross closed 4 months ago

FranECross commented 4 months ago

Description

Social media channel links under “Follow us” are broken on Benefit Hub page templates (e.g. https://www.va.gov/disability/) This can be seen it on healthcare and disability.

image.png

Steps to Reproduce

  1. Go to https://www.va.gov/disability/
  2. Scroll down to the 'Follow us' area on right of screen
  3. Click the Veterans Administration Facebook
  4. Click any of the other social media links

Engineering notes / background

Analytics considerations

Quality / testing notes

Acceptance criteria

jilladams commented 4 months ago

Noting: discussed with Dave / Fran in product sync that this missed today's daily deploy, and it doesn't warrant an OOB deploy. When this does deploy, we need to let Dave know so he can let VHA folks know it's fixed in prod.

@randimays do you have a sense from your work here what happened? Curious if this was a regression that happened during icon work, or something else acted on these links.

Also noting: Dave is curious about whether the broken link checker should have caught these, and I noted that they do not appear in the #content-broken-links channel. Might be because they're hard coded in templates and don't originate in Drupal content, not sure.

randimays commented 4 months ago

@jilladams This was a regression caused by va-icon updates. I was fairly certain I'd clicked those links to check when I updated the icons, but somehow I missed them. The PR was a fix for a syntax issue.

jilladams commented 4 months ago

Asked CMS whether the broken link checker should be checking links like this: https://dsva.slack.com/archives/CT4GZBM8F/p1717446503569169

randimays commented 4 months ago

Successfully validated the fix in production. Closing!