Skip to content

docs(all): full-permission Actors approval#2479

Open
tobice wants to merge 6 commits intomasterfrom
tobik/docs/full-permission-actors-approval
Open

docs(all): full-permission Actors approval#2479
tobice wants to merge 6 commits intomasterfrom
tobik/docs/full-permission-actors-approval

Conversation

@tobice
Copy link
Copy Markdown
Contributor

@tobice tobice commented Apr 29, 2026

We are about to launch forced approvals when running full-permission Actors. This PRs adds related documentation.

It also:

  • Updates outdated screenshots and unifies their location.
  • Makes small adjustments to the structure.
  • Fixes language here and there based on Claude's feedback (where it felt reasonable).

@github-actions github-actions Bot added the t-core-services Issues with this label are in the ownership of the core services team. label Apr 29, 2026
@apify-service-account
Copy link
Copy Markdown

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

@tobice tobice requested review from TC-MO, nmanerikar and raethlo and removed request for TC-MO April 29, 2026 13:44
@apify-service-account
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Contributor

@nmanerikar nmanerikar 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!
Couple of small non-blocking comments.

Comment thread sources/platform/actors/development/permissions/index.md Outdated
Copy link
Copy Markdown
Contributor

@raethlo raethlo 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 👍🏻

Copy link
Copy Markdown
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

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

Nothing major to fix, just some minor wording & a couple of opportunistic fixes

Comment thread sources/platform/actors/development/permissions/index.md Outdated
Comment thread sources/platform/actors/development/permissions/index.md Outdated
Comment thread sources/platform/actors/development/permissions/index.md Outdated
Comment thread sources/platform/actors/development/permissions/index.md Outdated
Comment thread sources/platform/actors/development/permissions/index.md Outdated
Comment thread sources/platform/actors/running/permissions.md Outdated
Co-authored-by: Nishad Manerikar <nmanerikar@yahoo.com>
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@tobice tobice left a comment

Choose a reason for hiding this comment

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

Thanks everyone for the review!

@apify-service-account
Copy link
Copy Markdown

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

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

Labels

t-core-services Issues with this label are in the ownership of the core services team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants