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

Siteimprove new gem #1374

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Siteimprove new gem #1374

wants to merge 4 commits into from

Conversation

KludgeKML
Copy link
Contributor

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

- We can't get the title of a broken link direct from
  Siteimprove at the moment, so workaround here is to
  load the content item from the Content Store and parse
  the body to extract the link title. Our links are
  simple enough to do this with a regex, but we might
  want to consider proper DOM parsing with nokogiri or
  similar if we see problems in the future
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant