Skip to content

feat: add support for maxItems in run options#330

Merged
B4nan merged 6 commits into
masterfrom
feature/add-max-items
Mar 9, 2023
Merged

feat: add support for maxItems in run options#330
B4nan merged 6 commits into
masterfrom
feature/add-max-items

Conversation

@novotnyj

@novotnyj novotnyj commented Mar 7, 2023

Copy link
Copy Markdown
Member

This adds support for optional maxItems param to start and call methods on actors and tasks.

This param will be used by pay per result actors to limit max items that will be charged to customer.

Should be merged after https://github.com/apify/apify-worker/pull/734 is released.

@github-actions github-actions Bot added the t-c&c Team covering store and finance matters. label Mar 7, 2023
@novotnyj novotnyj requested review from mhamas and vladfrangu March 7, 2023 13:29
Comment thread src/resource_clients/actor.ts Outdated
Comment thread test/actors.test.js Outdated
@novotnyj novotnyj requested a review from mhamas March 8, 2023 08:17
@B4nan B4nan changed the title feat: Add support for maxItems run options feat: add support for maxItems in run options Mar 9, 2023
@B4nan B4nan merged commit 9442187 into master Mar 9, 2023
@B4nan B4nan deleted the feature/add-max-items branch March 9, 2023 11:24
@B4nan

B4nan commented Mar 9, 2023

Copy link
Copy Markdown
Member

Should be merged after https://github.com/apify/apify-worker/pull/734 is released.

I hope it won't mind I merged this @novotnyj, will hold the stable release before that PR is merged.

@novotnyj

novotnyj commented Mar 9, 2023

Copy link
Copy Markdown
Member Author

It's ok, it won't break anything🙂

@stetizu1 stetizu1 removed the t-c&c Team covering store and finance matters. label Apr 10, 2026
@github-actions github-actions Bot added t-billing Issues owned by the billing team. tested Temporary label used only programatically for some analytics. labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-billing Issues owned by the billing team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants