Skip to content
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

docs: add new organisation readme #4

Merged
merged 4 commits into from
Dec 13, 2024
Merged

docs: add new organisation readme #4

merged 4 commits into from
Dec 13, 2024

Conversation

souravjain540
Copy link
Collaborator

adding new organisation readme.

@netmilk
Copy link

netmilk commented Dec 11, 2024

Thank you, @souravjain540!

The only two things that come to my mind are:

  • Crawlee py + js lines have a significantr repetition. What about squishing them into one line to align with the umbrella Crawlee page strategy and refer to "Crawlee for JavaScript" and "Crawlee for Python"? We could save some vertical space.

    For example:
    Crawlee: A web scraping and browser automation library for Node.js and Python to build reliable scrapers.

  • I'd consider mentionioning Actor Creation (authoring) in the Learn section more specifically. That's what we want to stress in the communication to the developer persona the most.

But it looks awesome otherwise and I'm leaving all that up to your consideration. :)

@souravjain540 souravjain540 marked this pull request as ready for review December 11, 2024 08:22
@souravjain540
Copy link
Collaborator Author

done @netmilk!

<li>Learn how to build Actors on Apify Store and monetize your dev skills through <a href="https://docs.apify.com/">Apify Docs</a> 📚 </li>
<li>Find tutorials and blogs on Apify and web scraping at the <a href="https://blog.apify.com/">Apify blog</a> 📝</li>
<li>View our livestreams and video content at the <a href="https://youtube.com/c/Apify">Apify YouTube channel</a> 📺</li>
<li>Find hundreds of ready-made Actors for your web scraping or automation project. Or create and <a href="https://apify.com/store/">publish your own.</a> 💻</li>
Copy link

Choose a reason for hiding this comment

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

Suggested change
<li>Find hundreds of ready-made Actors for your web scraping or automation project. Or create and <a href="https://apify.com/store/">publish your own.</a> 💻</li>
<li>Find hundreds of ready-made Actors for your web scraping or automation project. Or <a href="https://docs.apify.com/platform/actors/development/quick-start/locally">create</a> and <a href="https://apify.com/store/">publish your own.</a> 💻</li>

@jancurn
Copy link
Member

jancurn commented Dec 11, 2024

Thanks for that. Few small comments: please let's add proxy chain, it's one of our most popular repos.

The hyperlinks contain also ":" (e.g. fingerprint-suite: ), pls let's format it correctly.

@jancurn
Copy link
Member

jancurn commented Dec 11, 2024

Also "monetize your dev skills through Apify Docs" doesn't make much sense, each bullet point should communicate one thing: find ready-made Actors, learn web scraping, build your Actors, publish & monetize them, learn more

@souravjain540
Copy link
Collaborator Author

@jancurn done with changes, let me know if it is okay to merge now :)

Copy link
Member

@jancurn jancurn left a comment

Choose a reason for hiding this comment

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

Cool, go for it

@souravjain540 souravjain540 merged commit 193c418 into main Dec 13, 2024
1 check passed
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.

3 participants