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

[Connector] Crowdin events are not well triggered #2655

Open
srenault-meeds opened this issue Dec 9, 2024 · 1 comment
Open

[Connector] Crowdin events are not well triggered #2655

srenault-meeds opened this issue Dec 9, 2024 · 1 comment

Comments

@srenault-meeds
Copy link
Member

Steps to reproduce

Steps to reproduce the behavior:

  1. Add an action with event: "Approve Translation"
  2. Add an action with event: "Translation approved"
  3. Add a translation with user Translator
  4. Approve a translation with user Proofreader
  5. See error:
    • user Translator's action is listed in the "Approve translation"
    • event "Translation approved" is not collected

Video: https://youtu.be/4XeKK97YJtI

Expected behavior

Events are well captured

@AzmiTouil
Copy link
Contributor

The suggestion.approved webhook from Crowdin no longer provides the proofreader information in the payload. Instead, it only includes the translator details. Previously, the proofreader info was directly available in the payload, and the translator info could be retrieved from realizationService if related suggestion.added events were stored on the platform.

Crowdin Support Response:

  • The webhook behavior is expected and standardized to include only the translator info.

  • Webhooks are notifications and differ from APIs, which provide more detailed data.

  • To get proofreader details, you must call the Crowdin API after receiving the webhook.

  • No changes to webhook logic are planned.

  • API Limitation: Maximum of 20 simultaneous API requests per account. Exceeding this limit returns a 429 Too Many Requests error.

  • Recommendation: Modify your pipeline to make API calls after receiving the webhook to retrieve proofreader information.

https://community.crowdin.com/t/missing-proofreader-details-in-the-suggested-approved-webhook-payload/11211/7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Doing
Development

No branches or pull requests

2 participants