Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update monetize your Actor section #1337

Merged
merged 7 commits into from
Dec 12, 2024

Conversation

TC-MO
Copy link
Contributor

@TC-MO TC-MO commented Dec 10, 2024

rework structure
rephrase for better understanding & conciseness
add details tag to handle examples
add new admonitions to highlight important information

rework structure
rephrase for better understanding & conciseness
add details tag to handle examples
add new admonitions to highlight important information
@TC-MO TC-MO added documentation Improvements or additions to documentation. t-academy Issues related to Web Scraping and Apify academies. t-docs Issues owned by technical writing team. labels Dec 10, 2024
@TC-MO TC-MO self-assigned this Dec 10, 2024
@TC-MO TC-MO requested a review from honzajavorek as a code owner December 10, 2024 12:28
new structure
added details & summary tags for examples
added admonitions
rewrites for clarity & conciseness
@TC-MO TC-MO changed the title docs: update monetizy your Actor section docs: update monetize your Actor section Dec 10, 2024
Copy link
Member

@metalwarrior665 metalwarrior665 left a comment

Choose a reason for hiding this comment

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

Lot of these things now live in docs, right? So it makes sense to mostly remove stuff. We should try to honor the split of docs = what, academy = how.

This will need another rewrite once PPE is officially released. We will probably deprecate PPR and discourage rental. And then we have to agree where we will list the best practices. If there is enough in docs and SDK handles lot, then maybe we don't need any tutorial.

Comment on lines 41 to 57
| Service | Unit price |
|:---|:---|
| Compute unit | **$0.4** / CU |
| Residential proxies | **$13** / GB |
| SERPs proxy | **$3** / 1,000 SERPs |
| Data transfer - external | **$0.20** / GB |
| Data transfer - internal | **$0.05** / GB |
| Dataset - timed storage | **$1.00** / 1,000 GB-hours |
| Dataset - reads | **$0.0004** / 1,000 reads |
| Dataset - writes | **$0.005** / 1,000 writes |
| Key-value store - timed storage | **$1.00** / 1,000 GB-hours |
| Key-value store - reads | **$0.005** / 1,000 reads |
| Key-value store - writes | **$0.05** / 1,000 writes |
| Key-value store - lists | **$0.05** / 1,000 lists |
| Request queue - timed storage | **$4.00** / 1,000 GB-hours |
| Request queue - reads | **$0.004** / 1,000 reads |
| Request queue - writes | **$0.02** / 1,000 writes |
| Key-value store - reads | **$0.005** / 1,000 reads |
| Key-value store - writes | **$0.05** / 1,000 writes |
| Key-value store - lists | **$0.05** / 1,000 lists |
| Request queue - timed storage | **$4.00** / 1,000 GB-hours |
| Request queue - reads | **$0.004** / 1,000 reads |
| Request queue - writes | **$0.02** / 1,000 writes |
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: I find this harder to read in source code

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added more spacing to make it more readable in the source

@TC-MO TC-MO merged commit d75b4e7 into master Dec 12, 2024
8 checks passed
@TC-MO TC-MO deleted the update-academy/monetizing-your-actor branch December 12, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation. t-academy Issues related to Web Scraping and Apify academies. t-docs Issues owned by technical writing team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants