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(enterprise): App-based Workflow Step #200

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rohalskyy
Copy link
Collaborator

No description provided.

Copy link

vercel bot commented Feb 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 10, 2025 10:26pm

Copy link
Member

@andrii-bodnar andrii-bodnar left a comment

Choose a reason for hiding this comment

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

This article is missing a very important part - a reference to the developer portal article with information about the application module (not described) and information on how to create such an application.

Comment on lines +64 to +74
Depending on how the app is implemented, you might see different configuration options for an app-based workflow step, or sometimes none at all. For example:

* **Configurable Steps** – Some app-based steps offer configuration options in the workflow editor. For example, the **Schedule Delay** parameter as in the **Cron Delay** workflow step.
<Image src={configurableStep} alt="Configurable Workflow Step" class="width-sm m-auto" />

* **Pre-configured Steps** – Other steps might have all necessary settings predefined by the app (e.g., delay strings until the file is fully translated). In these cases, no additional fields appear in the workflow editor as in the **Full Translation Delay** workflow step.
<Image src={preconfiguredStep} alt="Pre-configured Workflow Step" class="width-sm m-auto" />

If your step provides configuration fields, they typically appear in the step's settings panel after you drag it into the workflow.

These fields may include anything from AI prompts and allowed actions (e.g., approving or deleting translations, adding translation suggestions or comments), API tokens, delay time, or delay conditions — essentially any options the app developer has made available. If there are no visible configuration options, the app is likely handling all necessary setup internally.
Copy link
Member

Choose a reason for hiding this comment

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

This section is too specific to the applications mentioned. I would suggest making it more general without mentioning the exact apps. That's the responsibility of the specific app listing in the Crowdin Store, as well as screenshots.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I can make this section more general by removing the screenshots that show specific app-based steps and by omitting references to specific apps in the text. Would that work?

Copy link
Member

Choose a reason for hiding this comment

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

Yes. I would also suggest expanding the description for these apps in the Crowdin Store using this information and screenshots.

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.

2 participants