Skip to content

Conversation

@patrikbraborec
Copy link
Contributor

Summary

  • Reorganized documentation into a clearer three-tier structure (introduction, concepts, guides)
  • Added separate pages for overview and installation in the introduction section
  • Streamlined the quick start guide to focus on essential concepts
  • Added new code examples for calling actors, pagination, and streaming resources

Test plan

  • Review the new documentation structure
  • Verify all internal links work correctly
  • Check that the sidebar navigation is logical and easy to follow
  • Test all code examples (both async and sync variants)
  • Ensure the quick start guide provides a clear path for new users

🤖 Generated with Claude Code

Reorganized documentation into a clearer three-tier structure:
- 01_introduction: Separate pages for overview, installation, and quick start
- 02_concepts: Core concepts including client architecture
- 03_guides: Practical guides for common use cases with code examples

Added new code examples for:
- Calling actors (async/sync)
- Pagination patterns (async/sync)
- Streaming resources (async/sync)

Streamlined the quick start guide to focus on essential concepts and moved detailed examples to the guides section.

Updated sidebar navigation and Docusaurus config to reflect the new structure.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added the t-c&c Team covering store and finance matters. label Jan 16, 2026
@codecov
Copy link

codecov bot commented Jan 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.01%. Comparing base (88af251) to head (7c9134f).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #579   +/-   ##
=======================================
  Coverage   76.01%   76.01%           
=======================================
  Files          42       42           
  Lines        2468     2468           
=======================================
  Hits         1876     1876           
  Misses        592      592           
Flag Coverage Δ
integration 68.96% <ø> (ø)
unit 64.58% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@B4nan B4nan requested a review from vdusek January 20, 2026 15:10
Copy link
Contributor

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

Just ideas, otherwise, LGTM

apify_client = ApifyClientAsync(TOKEN)

# Initialize the dataset client
dataset_client = apify_client.dataset('dataset-id')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it would be more practical to retrieve the dataset client by dataset's name?

apify_client = ApifyClient(TOKEN)

# Initialize the dataset client
dataset_client = apify_client.dataset('dataset-id')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it would be more practical to retrieve the dataset client by dataset's name?


async def main() -> None:
apify_client = ApifyClientAsync(TOKEN)
run_client = apify_client.run('MY-RUN-ID')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can have a module-level constant MY_RUN_ID, similar to how we do it for TOKEN.


def main() -> None:
apify_client = ApifyClient(TOKEN)
run_client = apify_client.run('MY-RUN-ID')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can have a module-level constant MY_RUN_ID, similar to how we do it for TOKEN.

apify_client = ApifyClient(TOKEN)

# Initialize the dataset client
dataset_client = apify_client.dataset('dataset-id')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can have a module-level constant DATASET_NAME, similar to how we do it for TOKEN.

apify_client = ApifyClientAsync(TOKEN)

# Initialize the dataset client
dataset_client = apify_client.dataset('dataset-id')
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can have a module-level constant DATASET_NAME, similar to how we do it for TOKEN.

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.

3 participants