-
Notifications
You must be signed in to change notification settings - Fork 141
docs: Make titles stand on their own #1751
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
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,12 @@ | ||
| --- | ||
| title: Development | ||
| 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 | ||
| --- | ||
|
|
||
| # Actor development | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is unnecessary, if this is to be the same as |
||
|
|
||
| **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.** | ||
|
|
||
| --- | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -124,4 +124,4 @@ You can also navigate Apify Console via keyboard shortcuts. | |
| | [Proxy](/platform/proxy)| View your proxy usage & credentials | | ||
| | [Storage](/platform/storage)| View stored results of your runs in various data formats. | | ||
| | [Billing](/platform/console/billing)| Billing information, statistics and invoices. | | ||
| | [Settings](/platform/console/settings)| Settings of your account. | | ||
| | [Settings](/platform/console/settings)| Console settings of your account. | | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the point of that change? The doc is within Console section, and this refers to Account settings. Moreover this is not specific Console settings , rather Apify acc settings.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Agree, this wasn't needed |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| --- | ||
| title: Settings | ||
| title: Console settings | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not needed as explained above. This is within Console section.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hey @TC-MO, IMO it makes total sense to make the page titles work on their own, so that you don't need to look in the left menu to figure what you're looking at. This change also makes the open graph images work better, and provides better context for LLMs who also don't see the menu structure.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. On one hand I completely agree that titles should work as standalone context for open graph and LLMs. That being said isn't this page about general Apify account settings rather than specific "Console settings"? It lives within the Console section, but it covers account-wide configurations. This seems similar to the case with the table mentioned above. Moreover if you take a look at the UI labels within Console, I deliberately chose these titles so they also reflect the naming conventions from Console.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fair enough, "Account settings" is a better name |
||
| description: Learn about the Apify Console settings for the account, integrations, organization management, and notification preferences. | ||
| sidebar_position: 4 | ||
| category: platform | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| --- | ||
| title: Actor-to-Actor | ||
| title: Actor-to-Actor integration | ||
| description: Learn how to integrate with other Actors and tasks. | ||
| sidebar_label: Actor-to-Actor | ||
| sidebar_label: Actor-to-Actor integration | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The sidebar change was not necessary and actually it makes it worse |
||
| slug: /integrations/actors | ||
| --- | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| --- | ||
| title: Usage | ||
| title: Usage of storage | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For better syntactic parallelism with the
|
||
| description: Learn how to effectively use Apify's storage options. Understand key aspects of data retention, rate limiting, and secure sharing. | ||
| sidebar_position: 9.1 | ||
| category: platform | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don' think this is a good change, it already is inferred from the structure that this development is in relation to Actors. I would leave original title. Also this changes the title on the right side, which can get cluttered unnecessarily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with keeping the sidebar label, but IMO this change makes sense, so that you know what you're looking at without having to look at the menu