Skip to content

Conversation

Kanishkavijay39
Copy link
Contributor

For sandboxes we don't have data on ahref so trying to running audit on some defined urls
Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes
  • If data sources for any opportunity has been updated/added, please update the wiki for same opportunity.

Related Issues

Thanks for contributing!

Kanishka added 3 commits September 7, 2025 19:28
- Update submitForScraping to combine top pages and included URLs
- Update runAuditAndGenerateSuggestions to use combined URL list
- Add proper error handling when no URLs are found
- Follow same pattern as metatags handler for consistency
- Ensure audit works even when ahrefs top pages are unavailable
- Temporarily using 'meta-tags' config for testing purposes

This change ensures the structured data audit can run successfully even when
ahrefs top pages are not available by falling back to site-configured included URLs,
matching the behavior of the metatags handler. For Sandbox.
Copy link

github-actions bot commented Sep 7, 2025

This PR will trigger a minor release when merged.

- Fix trailing spaces and indentation issues
- Install missing @babel/eslint-parser dependency
- All tests still passing with 100% coverage
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