Skip to content

Comments

fix: completionUrl not sent to content scraper and introduce skip property#685

Merged
dzehnder merged 3 commits intomainfrom
fix-scraper-payload
Apr 17, 2025
Merged

fix: completionUrl not sent to content scraper and introduce skip property#685
dzehnder merged 3 commits intomainfrom
fix-scraper-payload

Conversation

@dzehnder
Copy link
Contributor

@dzehnder dzehnder commented Apr 10, 2025

Adding missing completionUrl so the audit worker can be triggered back after the scraper executes.

Adding allowCache property for both the importer and scraper to return the already existing content instead of triggering an import/scrape again for every audit.

@github-actions
Copy link

This PR will trigger a minor release when merged.

@dzehnder dzehnder changed the title fix: completionUrl not sent to content scraper fix: completionUrl not sent to content scraper and introduce skip property Apr 15, 2025
@dzehnder dzehnder self-assigned this Apr 15, 2025
@dzehnder dzehnder added the bug Something isn't working label Apr 15, 2025
@dzehnder dzehnder merged commit 4d89a29 into main Apr 17, 2025
7 checks passed
@dzehnder dzehnder deleted the fix-scraper-payload branch April 17, 2025 08:11
solaris007 pushed a commit that referenced this pull request Apr 17, 2025
# [@adobe/spacecat-shared-data-access-v2.15.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v2.14.1...@adobe/spacecat-shared-data-access-v2.15.0) (2025-04-17)

### Bug Fixes

* completionUrl not sent to content scraper and introduce skip property ([#685](#685)) ([4d89a29](4d89a29))

### Features

* Add contentAiConfig validation to Config class ([#686](#686)) ([077fbbd](077fbbd))
@solaris007
Copy link
Member

🎉 This PR is included in version @adobe/spacecat-shared-data-access-v2.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants