Conversation
|
Too many files changed for review. ( |
LogDetails |
There was a problem hiding this comment.
31 issues found across 124 files
Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="packages/twenty-front/src/locales/ja-JP.po">
<violation number="1" location="packages/twenty-front/src/locales/ja-JP.po:12818">
P3: The new "webhook" string is left untranslated in the Japanese locale, so users will see the English text instead of a localized label.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P2: The new webhook error "A webhook with this configuration already exists." was added to the Arabic locale with an empty msgstr, so Arabic-speaking users will now see the English fallback. Please provide the Arabic translation.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P3: "Webhook not found" was added without an Arabic translation, so Arabic-speaking users will see an English error despite this being the Arabic locale file.</violation>
<violation number="3" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P3: "Target URL is required" needs an Arabic translation; leaving msgstr empty causes the Arabic locale to display the English fallback for this validation error.</violation>
<violation number="4" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P2: "Please provide a valid HTTP or HTTPS URL" no longer has an Arabic translation even though this locale previously translated the same message, so users regress to English.</violation>
<violation number="5" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P2: The replacement error message "Invalid target URL. Please provide a valid HTTP or HTTPS URL." lost its Arabic translation, causing a regression in localized UX.</violation>
<violation number="6" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po:176">
P2: "Invalid target URL provided" was added without an Arabic translation, so this validation message will display in English in the Arabic UI.</violation>
</file>
<file name="packages/twenty-front/src/locales/sr-Cyrl.po">
<violation number="1" location="packages/twenty-front/src/locales/sr-Cyrl.po:12818">
P3: Provide a Serbian translation for the newly added "webhook" string; leaving msgstr empty causes the locale to display the English fallback.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: The new webhook duplicate-configuration message has no zh-CN translation, so Chinese users will see English text.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: The newly added “Webhook not found” string needs a zh-CN translation; otherwise the locale shows English for this validator error.</violation>
<violation number="3" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: “Please provide a valid HTTP or HTTPS URL” no longer has a zh-CN msgstr, so users see English despite a prior translation existing.</violation>
<violation number="4" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: “Invalid webhook input.” was added without a zh-CN translation, breaking localization consistency.</violation>
<violation number="5" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: The extended target URL validation error lost its Chinese translation, causing zh-CN users to see English.</violation>
<violation number="6" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po:176">
P2: “Invalid target URL provided” was added without a zh-CN msgstr, so the translated locale falls back to English.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-PT.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-PT.po:1963">
P2: Provide the Portuguese translation for “Invalid target URL. Please provide a valid HTTP or HTTPS URL.”—otherwise users who previously saw this error in Portuguese now get English text.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-PT.po:2971">
P2: Restore the Portuguese translation for “Please provide a valid HTTP or HTTPS URL”; the previous localized text was removed and nothing replaced it, so the locale now falls back to English.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-BR.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-BR.po:176">
P3: Provide a translation for this new webhook-not-found string to avoid fallback English output.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-BR.po:176">
P3: Add the pt-BR translation for "Target URL is required".</violation>
<violation number="3" location="packages/twenty-server/src/engine/core-modules/i18n/locales/pt-BR.po:176">
P3: Provide the Portuguese translation for this new webhook duplication error so the locale stays fully translated.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/cs-CZ.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/cs-CZ.po:176">
P2: Add the Czech translation for “Please provide a valid HTTP or HTTPS URL” so the validator prompt remains localized.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/cs-CZ.po:176">
P2: Restore a Czech translation for “Invalid target URL. Please provide a valid HTTP or HTTPS URL.” so the user-facing validation message stays localized.</violation>
<violation number="3" location="packages/twenty-server/src/engine/core-modules/i18n/locales/cs-CZ.po:176">
P2: Translate “Target URL is required” to keep the required-field error localized.</violation>
</file>
<file name="packages/twenty-front/src/locales/fr-FR.po">
<violation number="1" location="packages/twenty-front/src/locales/fr-FR.po:12820">
P3: Provide a French translation for the newly added msgid "webhook" instead of leaving msgstr empty so the locale doesn’t fall back to English.</violation>
</file>
<file name="packages/twenty-front/src/locales/ar-SA.po">
<violation number="1" location="packages/twenty-front/src/locales/ar-SA.po:12816">
P3: The newly added “webhook” string is missing its Arabic translation, so the locale will fall back to English instead of showing localized text. Please provide the appropriate translation.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/es-ES.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/es-ES.po:176">
P2: The new webhook error string lost its Spanish translation, so Spanish users will now see the English message instead of the previous localized text.</violation>
</file>
<file name="packages/twenty-front/src/locales/ro-RO.po">
<violation number="1" location="packages/twenty-front/src/locales/ro-RO.po:12818">
P3: Provide a Romanian translation for the newly added "webhook" string; leaving msgstr empty results in a missing label in this locale.</violation>
</file>
<file name="packages/twenty-front/src/locales/uk-UA.po">
<violation number="1" location="packages/twenty-front/src/locales/uk-UA.po:12820">
P3: Provide a Ukrainian translation for the new "webhook" string; leaving msgstr empty makes the UI show the English word in the Ukrainian locale.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po:176">
P2: Restore the French translation for “Please provide a valid HTTP or HTTPS URL” so the validator error remains localized.</violation>
<violation number="2" location="packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po:176">
P2: Provide the French translation for “Invalid target URL. Please provide a valid HTTP or HTTPS URL.” so the webhook error remains localized.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/he-IL.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/he-IL.po:176">
P2: The new “Please provide a valid HTTP or HTTPS URL” string lost its Hebrew translation, so he‑IL users will now see English text instead of the previously available localization.</violation>
</file>
<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/fi-FI.po">
<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/fi-FI.po:176">
P3: Translate “Invalid webhook input.” so Finnish users don’t see the fallback English text.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P2: The new webhook error "A webhook with this configuration already exists." was added to the Arabic locale with an empty msgstr, so Arabic-speaking users will now see the English fallback. Please provide the Arabic translation.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po, line 176:
<comment>The new webhook error "A webhook with this configuration already exists." was added to the Arabic locale with an empty msgstr, so Arabic-speaking users will now see the English fallback. Please provide the Arabic translation.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "حدث خطأ في مجموعة العرض."
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| msgstr "" | |
| msgstr "يوجد ويب هوك بهذا التكوين بالفعل." |
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P2: "Please provide a valid HTTP or HTTPS URL" no longer has an Arabic translation even though this locale previously translated the same message, so users regress to English.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po, line 176:
<comment>"Please provide a valid HTTP or HTTPS URL" no longer has an Arabic translation even though this locale previously translated the same message, so users regress to English.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "حدث خطأ في مجموعة العرض."
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| msgstr "" | |
| msgstr "يرجى تقديم رابط صحيح بنظام HTTP أو HTTPS." |
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P2: The replacement error message "Invalid target URL. Please provide a valid HTTP or HTTPS URL." lost its Arabic translation, causing a regression in localized UX.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po, line 176:
<comment>The replacement error message "Invalid target URL. Please provide a valid HTTP or HTTPS URL." lost its Arabic translation, causing a regression in localized UX.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "حدث خطأ في مجموعة العرض."
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| msgstr "" | |
| msgstr "عنوان URL للهدف غير صالح. يرجى تقديم رابط صحيح بنظام HTTP أو HTTPS." |
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P2: "Invalid target URL provided" was added without an Arabic translation, so this validation message will display in English in the Arabic UI.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/ar-SA.po, line 176:
<comment>"Invalid target URL provided" was added without an Arabic translation, so this validation message will display in English in the Arabic UI.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "حدث خطأ في مجموعة العرض."
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P2: The new webhook duplicate-configuration message has no zh-CN translation, so Chinese users will see English text.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/zh-CN.po, line 176:
<comment>The new webhook duplicate-configuration message has no zh-CN translation, so Chinese users will see English text.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "发生视图组错误。"
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| msgstr "" | |
| msgstr "具有此配置的 Webhook 已存在。" |
| #. js-lingui-id: Dq7Z5R | ||
| #: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts | ||
| msgid "webhook" | ||
| msgstr "" |
There was a problem hiding this comment.
P3: Provide a French translation for the newly added msgid "webhook" instead of leaving msgstr empty so the locale doesn’t fall back to English.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-front/src/locales/fr-FR.po, line 12820:
<comment>Provide a French translation for the newly added msgid "webhook" instead of leaving msgstr empty so the locale doesn’t fall back to English.</comment>
<file context>
@@ -12814,6 +12814,11 @@ msgstr ""
+#. js-lingui-id: Dq7Z5R
+#: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts
+msgid "webhook"
+msgstr ""
+
#. js-lingui-id: Tvdycs
</file context>
| msgstr "" | |
| msgstr "webhook" |
| #. js-lingui-id: Dq7Z5R | ||
| #: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts | ||
| msgid "webhook" | ||
| msgstr "" |
There was a problem hiding this comment.
P3: The newly added “webhook” string is missing its Arabic translation, so the locale will fall back to English instead of showing localized text. Please provide the appropriate translation.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-front/src/locales/ar-SA.po, line 12816:
<comment>The newly added “webhook” string is missing its Arabic translation, so the locale will fall back to English instead of showing localized text. Please provide the appropriate translation.</comment>
<file context>
@@ -12810,6 +12810,11 @@ msgstr "نحن في انتظار تأكيد من مزود الدفع لدينا
+#. js-lingui-id: Dq7Z5R
+#: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts
+msgid "webhook"
+msgstr ""
+
#. js-lingui-id: Tvdycs
</file context>
| #. js-lingui-id: Dq7Z5R | ||
| #: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts | ||
| msgid "webhook" | ||
| msgstr "" |
There was a problem hiding this comment.
P3: Provide a Romanian translation for the newly added "webhook" string; leaving msgstr empty results in a missing label in this locale.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-front/src/locales/ro-RO.po, line 12818:
<comment>Provide a Romanian translation for the newly added "webhook" string; leaving msgstr empty results in a missing label in this locale.</comment>
<file context>
@@ -12812,6 +12812,11 @@ msgstr ""
+#. js-lingui-id: Dq7Z5R
+#: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts
+msgid "webhook"
+msgstr ""
+
#. js-lingui-id: Tvdycs
</file context>
| msgstr "" | |
| msgstr "webhook" |
| #. js-lingui-id: Dq7Z5R | ||
| #: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts | ||
| msgid "webhook" | ||
| msgstr "" |
There was a problem hiding this comment.
P3: Provide a Ukrainian translation for the new "webhook" string; leaving msgstr empty makes the UI show the English word in the Ukrainian locale.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-front/src/locales/uk-UA.po, line 12820:
<comment>Provide a Ukrainian translation for the new "webhook" string; leaving msgstr empty makes the UI show the English word in the Ukrainian locale.</comment>
<file context>
@@ -12814,6 +12814,11 @@ msgstr ""
+#. js-lingui-id: Dq7Z5R
+#: src/modules/metadata-error-handler/hooks/useMetadataErrorHandler.ts
+msgid "webhook"
+msgstr ""
+
#. js-lingui-id: Tvdycs
</file context>
| msgstr "" | |
| msgstr "вебхук" |
| #. js-lingui-id: SMhyGZ | ||
| #: src/engine/metadata-modules/webhook/webhook.exception.ts | ||
| msgid "A webhook with this configuration already exists." | ||
| msgstr "" |
There was a problem hiding this comment.
P3: Translate “Invalid webhook input.” so Finnish users don’t see the fallback English text.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/fi-FI.po, line 176:
<comment>Translate “Invalid webhook input.” so Finnish users don’t see the fallback English text.</comment>
<file context>
@@ -170,6 +170,11 @@ msgstr "Näkymäryhmässä tapahtui virhe."
+#. js-lingui-id: SMhyGZ
+#: src/engine/metadata-modules/webhook/webhook.exception.ts
+msgid "A webhook with this configuration already exists."
+msgstr ""
+
#. js-lingui-id: Bk+qoW
</file context>
| msgstr "" | |
| msgstr "Webhookin syöte on virheellinen." |
Created by Github action