docs(developer): document the Workflow Step Type module comprehensively - #875
Merged
andrii-bodnar merged 3 commits intoAug 6, 2026
Merged
Conversation
|
Someone is attempting to deploy a commit to the Crowdin Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Significantly expand the workflow-step-type module documentation to cover the crowdin_agent authentication requirement, the agent user concept, the token flow, port semantics, string statuses, webhook delivery guarantees, settings lifecycle callbacks, a recommended implementation flow, limitations, and troubleshooting. Also: - document the crowdin_agent authentication type in the App Descriptor article and reference it from the Security article - describe the agent user and failed strings recovery in the App-based Workflow Step article Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
geeksman
force-pushed
the
docs/workflow-step-type-module
branch
from
August 6, 2026 10:47
1e2b812 to
604eca0
Compare
… Type docs - correct what is validated at installation and how a missing Webhook module actually surfaces (silent non-delivery first, validation error only on later workflow saves) - document the JWT-signed settings callbacks, per-minute webhook batching, agent name/username handling, and the agentId Installed event property - clarify the INCOMPLETE status and quote real validation messages in Troubleshooting - restructure the module article: section order aligned with sibling articles, Limitations folded into existing sections, simple tables converted to Markdown - use "bot" terminology in the Enterprise article to match the UI - update the Installed event note in the App Installation article Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
Code reviewFound 2 issues:
docs/src/content/docs/developer/crowdin-apps/app-descriptor.mdx Lines 259 to 261 in 3199a5d
docs/src/content/docs/developer/modules/other/workflow-step-type.mdx Lines 650 to 658 in 3199a5d 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
rohalskyy
approved these changes
Aug 6, 2026
… fix the installed event description - use the sample manifest's declared output ports (translated/untranslated) in the Update String Status request and in both response examples - mention crowdin_agent in the installed event description in the App Descriptor Events table Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
rohalskyy
approved these changes
Aug 6, 2026
andrii-bodnar
approved these changes
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
workflow-step-typemodule requires thecrowdin_agentauthentication type, but this constraint — and thecrowdin_agenttype itself — was not documented. Installing an app with any other authentication type fails manifest validation with "Only crowdin_agent authentication type is allowed for workflow-step-type module type", and there was no public documentation explaining why or how to configure agent authentication.This PR significantly expands the Workflow Step Type module documentation and documents the
crowdin_agentauthentication type across related articles. It also incorporates the technical review feedback (accuracy corrections, section restructuring, and terminology alignment).developer/modules/other/workflow-step-type.mdx(main rewrite)crowdin_agent, the agent (bot) user concept, theagentdescriptor block, and the full token flow (grant_type: crowdin_agent,agentIdfrom the Installed event)initialis input-only, max 2 outputsreplace-only PATCH semantics, examples aligned with the sample manifest's declared ports), JWT-signed settings lifecycle callbacks (project and template payloads), Recommended Implementation Flow, and a Troubleshooting table quoting the real validation messageslogo/description)enterprise/.../app-based-workflow-step.mdxdeveloper/crowdin-apps/app-descriptor.mdx,security.mdx, andinstallation.mdxcrowdin_agentdocumented as a third authentication type with links to the module pagecrowdin_appandcrowdin_agent(payload additionally carriesagentId)Notes
astro check(0 errors) and a fullastro build— all pages build and every internal link is valid🤖 Generated with Claude Code