Skip to content

feat(docs): Pay-per-event with usage not included disclaimer#2229

Open
stetizu1 wants to merge 5 commits intomasterfrom
user-pricing-page
Open

feat(docs): Pay-per-event with usage not included disclaimer#2229
stetizu1 wants to merge 5 commits intomasterfrom
user-pricing-page

Conversation

@stetizu1
Copy link
Contributor

@stetizu1 stetizu1 commented Feb 5, 2026

Resolves: https://github.com/apify/apify-core/issues/24692

Adds PPE+U to docs
Swap screenshot for current version in console

Preview for this PR was built for commit 4eeac644 and is ready at https://pr-2229.preview.docs.apify.com


Note

Low Risk
Documentation-only edits (wording and screenshots) with no runtime or API behavior changes.

Overview
Updates the Store pricing-model docs to clarify Pay per event semantics, explicitly calling out that some Actors may bill platform usage separately and adding a caution block plus an example screenshot.

Reorders/replaces screenshots in the Rental and Pay per event sections to match the current Console/Store UI and adjusts related wording/headings for clarity.

Written by Cursor Bugbot for commit 2bf7b28. Configure here.

@stetizu1 stetizu1 requested a review from TC-MO as a code owner February 5, 2026 17:20
@stetizu1 stetizu1 requested review from TC-MO, Copilot and mhamas and removed request for TC-MO February 5, 2026 17:20
@github-actions github-actions bot added the t-c&c Team covering store and finance matters. label Feb 5, 2026
Copy link
Contributor

Copilot AI 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 overview

This PR updates the documentation for pay per event Actor pricing to clarify that some Actors charge platform usage separately (PPE+U model). It also replaces an outdated screenshot for rental Actors with the current console version.

Changes:

  • Clarified pay per event pricing model description in the pricing models list
  • Added a caution admonition and FAQ entry explaining that some pay per event Actors charge platform usage separately
  • Removed outdated rental Actor trial screenshot and consolidated to single screenshot
  • Added new screenshot showing pay per event with usage pricing example

Reviewed changes

Copilot reviewed 1 out of 6 changed files in this pull request and generated 1 comment.

File Description
sources/platform/actors/running/store.md Updated pay per event pricing documentation to explain PPE+U model and reorganized rental Actor screenshots
sources/platform/actors/running/images/store/rental-actor-trial-example.png Removed outdated screenshot

@apify-service-account
Copy link

Preview for this PR was built for commit 2bf7b28d and is ready at https://pr-2229.preview.docs.apify.com!

@stetizu1 stetizu1 changed the title swap screenshots for current state feat(docs): Pay-per-event with usage not included disclaimer Feb 6, 2026
@stetizu1
Copy link
Contributor Author

stetizu1 commented Feb 6, 2026

Note: in followup we should

  • remove PPR
  • Maybe? move PPE to first place? I know we will still have slightly more rentals, even after PPR->E migration, but this should be our future main pricing model, we should probably "sell" it first

@apify-service-account
Copy link

Preview for this PR was built for commit 582a2883 and is ready at https://pr-2229.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 2dd61c2c and is ready at https://pr-2229.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 84661747 and is ready at https://pr-2229.preview.docs.apify.com!

Copy link
Contributor Author

@stetizu1 stetizu1 Feb 6, 2026

Choose a reason for hiding this comment

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

Removed
Image
unused

@apify-service-account
Copy link

Preview for this PR was built for commit 4eeac644 and is ready at https://pr-2229.preview.docs.apify.com!

Copy link
Contributor Author

@stetizu1 stetizu1 Feb 6, 2026

Choose a reason for hiding this comment

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

Removed
Image
unused

Copy link
Contributor Author

@stetizu1 stetizu1 Feb 6, 2026

Choose a reason for hiding this comment

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

Removed
Image
unused

Copy link
Contributor Author

@stetizu1 stetizu1 Feb 6, 2026

Choose a reason for hiding this comment

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

I actually only replaced this one, but for the sake of consistency the new one has _ instead of -... 😅

Image

Copy link
Contributor Author

@stetizu1 stetizu1 Feb 6, 2026

Choose a reason for hiding this comment

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

Removed
Image
also obviously unused

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 removed this one. it does not look like this any longer and any replacement would be just duplicite with existing solution

Image

Copy link
Contributor

Copilot AI 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 overview

Copilot reviewed 1 out of 15 changed files in this pull request and generated no new comments.

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

Labels

t-c&c Team covering store and finance matters.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants