Skip to content

feat: web fetch action - #60

Merged
protoss70 merged 6 commits into
masterfrom
feat/web-fetch-action
Sep 22, 2026
Merged

feat: web fetch action#60
protoss70 merged 6 commits into
masterfrom
feat/web-fetch-action

Conversation

@protoss70

@protoss70 protoss70 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Closes #49

Added new Web Fetch option. Deprecated old Scrape Single URL action via versioning.

  • Users who already had scrape single URL will see a deprecation notice.
  • Newer Apify nodes added will not see the Scrape Single URL action at all
Screenshot 2026-09-17 at 11 23 15

@protoss70 protoss70 self-assigned this Sep 17, 2026
@protoss70 protoss70 added the t-integrations Issues with this label are in the ownership of the integrations team. label Sep 17, 2026
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Sep 17, 2026
@github-actions

github-actions Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

AGENTS.md updated

  • "Adding an operation" step 2 pointed at an operations array that actors/index.ts no longer has — now names operationsV1/operationsV2

Committed to this PR. Review it like any other commit. Feel free to edit it.

drobnikj
drobnikj requested changes Sep 17, 2026
View reviewed changes

@drobnikj drobnikj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just two notes

Comment thread nodes/Apify/__tests__/utils/fixtures.ts
Comment thread nodes/Apify/resources/actors/web-fetch/execute.ts
@protoss70
protoss70 removed the request for review from apify-factory September 17, 2026 14:22
@protoss70

protoss70 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

@apify-factory Please implement the review changes requested

  • Check out existing tests, get inspired and implement testing coverage for the new action
  • Check out the existing timeout on dataset but dont set it less than 120 seconds!

@github-actions

github-actions Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

AGENTS.md updated

  • Web Fetch handles two error envelopes — noted the nested { error: { message, type } } body alongside the flat one

Committed to this PR. Review it like any other commit. Feel free to edit it.

@protoss70
protoss70 requested a review from drobnikj September 18, 2026 13:01
@protoss70
protoss70 force-pushed the feat/web-fetch-action branch from eb49cac to 45add67 Compare September 18, 2026 13:04
drobnikj
drobnikj approved these changes Sep 21, 2026
View reviewed changes

@drobnikj drobnikj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just one note, but pre approving

Comment thread nodes/Apify/resources/actors/web-fetch/execute.ts Outdated
@protoss70
protoss70 merged commit 77f2038 into master Sep 22, 2026
3 checks passed
@FilipMasar FilipMasar mentioned this pull request Oct 1, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drobnikj drobnikj drobnikj approved these changes
@JanHranicky JanHranicky JanHranicky approved these changes

Assignees

@protoss70 protoss70

Labels

t-integrations Issues with this label are in the ownership of the integrations team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

n8n: Add Web Fetch operation and deprecate Scrape single URL

5 participants

@protoss70 @drobnikj @JanHranicky @apify-service-account @apify-factory