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: create new telegram integration guide #825

Merged
merged 14 commits into from
Jan 23, 2024
Merged

Conversation

TC-MO
Copy link
Contributor

@TC-MO TC-MO commented Jan 11, 2024

add new screenshots for zapier & telegram integration within zapier
update outdated zapier screenshot
add new telegram integration guide

TC-MO added 3 commits January 11, 2024 16:49
add new screenshots for zapier & telegram integration within zapier
update outdated zapier screenshot
add new telegram integration guide
@TC-MO TC-MO requested a review from TheoVasilis January 11, 2024 15:58
@TC-MO TC-MO requested review from TheoVasilis and mtrunkat January 11, 2024 17:04
Copy link
Member

@mtrunkat mtrunkat left a comment

Choose a reason for hiding this comment

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

One thing is not clear to me

sources/platform/integrations/langchain.md Show resolved Hide resolved

:::info Available Actors & Tasks

Please note that only Actor runs or Actor task runs that you previously executed will be available to choose from.
Copy link
Member

@mtrunkat mtrunkat Jan 15, 2024

Choose a reason for hiding this comment

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

Itsn't this

Suggested change
Please note that only Actor runs or Actor task runs that you previously executed will be available to choose from.
Please note that only Actors or Actor tasks that you previously run will be available to choose from.

? You are choosing an Actor or Task not exact run, right?

But in fact, is this true? I think that you can now select any Actor or Task without running it previously. I think that previously we offered people to choose from previously run Actors but recently we added search for any own or public Actor.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, good catch it should have been Actor and Actor tasks not Actor run and Actor task runs.

Regarding the second point, has this changed within the last month since the publication of the video guide? I took the info from the video, and within it we have disclaimer that only Actors that were previously run will show up within Zapier when using Apify as trigger.

Copy link
Member

Choose a reason for hiding this comment

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

@drobnikj would know when this was changed - you updated the Zapier integration in autumn to be able to select any Actor, right? So it might be incorrect in the video which is not a big deal IMHO.

Copy link
Contributor Author

@TC-MO TC-MO Jan 18, 2024

Choose a reason for hiding this comment

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

I spoke with @drobnikj yesterday, when using Apify as a trigger you can only use previously run Actors or Actor tasks. So the information is correct both here and within the video.

@TC-MO TC-MO requested a review from mtrunkat January 17, 2024 09:44
@TC-MO TC-MO requested a review from drobnikj January 17, 2024 13:26
Copy link
Member

@drobnikj drobnikj left a comment

Choose a reason for hiding this comment

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

Just one typo on the screen to fix. Otherwise good 👍

@TC-MO TC-MO merged commit 6f56191 into master Jan 23, 2024
6 checks passed
@TC-MO TC-MO deleted the telegram-integration-guide branch January 23, 2024 14:01
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.

4 participants