Skip to content

Conversation

@patrikbraborec
Copy link
Contributor

@patrikbraborec patrikbraborec commented Aug 4, 2025

This is a follow up for the PR: #1751. Some titles need to improve as I did not wait for @TC-MO - sorry!

@patrikbraborec patrikbraborec requested a review from jancurn August 4, 2025 07:55
@patrikbraborec patrikbraborec requested a review from TC-MO as a code owner August 4, 2025 07:55
@apify-service-account
Copy link

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

Copy link
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.

One change requested, IIRC we spoke in the previous thread about keeping sidebar label unchanged and changing just the page title

title: Actor development
desc: Read about the technical part of building Apify Actors. Learn to define Actor inputs, build new versions, persist Actor state, and choose base Docker images.
sidebar_position: 7.4
slug: /actors/development
Copy link
Contributor

@TC-MO TC-MO Aug 4, 2025

Choose a reason for hiding this comment

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

Can we add sidebar_label: Development to frontmatter? I don't think placement matters much, but for consistency I would prefer under description

Copy link
Contributor

Choose a reason for hiding this comment

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

And based on this discussion can we get Account settings instead of Console settings

@patrikbraborec
Copy link
Contributor Author

@TC-MO thanks for the review. I updated the sidebar label based on your proposal.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Comment bugbot run to trigger another review on this PR

@apify-service-account
Copy link

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

@apify-service-account
Copy link

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

Copy link
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.

LGTM, I'll make separate PR for Account settings

@patrikbraborec patrikbraborec merged commit 050cc7a into master Aug 4, 2025
9 checks passed
@patrikbraborec patrikbraborec deleted the improve/docs-titles-fix branch August 4, 2025 11:12
daveomri pushed a commit to daveomri/apify-docs that referenced this pull request Sep 3, 2025
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.

5 participants