Skip to content

feat: add pay per result docs#518

Closed
mhamas wants to merge 6 commits into
masterfrom
feat/pay-per-result-docs
Closed

feat: add pay per result docs#518
mhamas wants to merge 6 commits into
masterfrom
feat/pay-per-result-docs

Conversation

@mhamas
Copy link
Copy Markdown
Contributor

@mhamas mhamas commented Feb 22, 2023

Basic documentation on what pay per result actors are. Note this is only first version. I think the good approach here is to get out something basic as soon as possible and iterate later.

For better reading (images cannot be easily rendered locally)
CleanShot 2023-02-22 at 16 33 17@2x

@github-actions github-actions Bot added the t-c&c Team covering store and finance matters. label Feb 22, 2023
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

@B4nan
Copy link
Copy Markdown
Member

B4nan commented Feb 22, 2023

Keep in mind this repo still has old content, I will move it on monday when I am back from vacation (pr is here already). So you should probably move this to the apify-docs-v2 repo.

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

@mhamas mhamas marked this pull request as ready for review February 22, 2023 13:09
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
Comment thread content/docs/actors/pay_per_result_actors.md Outdated
@mhamas mhamas requested review from jancurn and mtrunkat February 22, 2023 14:57
@B4nan
Copy link
Copy Markdown
Member

B4nan commented Feb 22, 2023

Let me say it one more time, this should be targetting the apify-docs-v2 repo or wait before #517 gets merged.

@mhamas
Copy link
Copy Markdown
Contributor Author

mhamas commented Feb 23, 2023

@B4nan I don't need to deploy it until next week, so it should be fine. I looked at v2 repo as well, but I wasn't sure where it would be the best to put this content and how it exactly interacts with the old content in this repo. Let's discuss next week when you are back, it's not urgent. Thx

Copy link
Copy Markdown
Member

@mtrunkat mtrunkat left a comment

Choose a reason for hiding this comment

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

Looks good. I plan to transform this section a lot but this will serve as a great base.

@mtrunkat
Copy link
Copy Markdown
Member

mtrunkat commented Mar 1, 2023

Closing in favor of #532

@mtrunkat mtrunkat closed this Mar 1, 2023
@B4nan B4nan deleted the feat/pay-per-result-docs branch November 25, 2025 10:49
@stetizu1 stetizu1 removed the t-c&c Team covering store and finance matters. label Apr 10, 2026
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.

6 participants