23tae commented on code in PR #68234:
URL: https://github.com/apache/airflow/pull/68234#discussion_r3397567407
##########
airflow-core/src/airflow/ui/public/i18n/locales/ko/dashboard.json:
##########
@@ -1,4 +1,6 @@
{
+ "deferredSlotsNotCounted": "슬롯 계산에서 제외된 연기된 슬롯: {{count}}개",
Review Comment:
```suggestion
"deferredSlotsNotCounted": "슬롯 계산에서 제외된 연기된 태스크: {{count}}개",
```
Although the key includes the word `Slots`, the English text is `"Deferred
not counted in slots"`, and the tooltip clarifies that this actually counts
`Deferred tasks`. Translating it as "연기된 태스크" avoids confusion and aligns
accurately with the tooltip description.
##########
airflow-core/src/airflow/ui/public/i18n/locales/ko/components.json:
##########
@@ -72,13 +73,15 @@
"files_other": "{{count}}개 파일"
},
"flexibleForm": {
+ "durationPlaceholder": "ISO 8601 형식의 기간 입력",
"placeholder": "값 선택",
"placeholderArray": "각 문자열을 새 줄에 입력하세요.",
"placeholderExamples": "입력하여 옵션 보기",
"placeholderMulti": "하나 또는 여러 값 선택",
"validationErrorArrayNotArray": "값은 배열이어야 합니다.",
"validationErrorArrayNotNumbers": "배열의 모든 요소는 숫자여야 합니다.",
"validationErrorArrayNotObject": "배열의 모든 요소는 객체여야 합니다.",
+ "validationErrorDuration": "ISO 8601 형식에 맞는 기간을 입력해 주세요.",
Review Comment:
```suggestion
"validationErrorDuration": "올바르지 않은 ISO 8601 기간 형식입니다.",
```
Other validation error messages in this file use a declarative sentence
(e.g., `"validationErrorRequired": "이 필드는 필수입니다."`). To keep the tone
consistent, changing this to a declarative form ("~입니다") would be better than a
request form ("~해 주세요").
##########
airflow-core/src/airflow/ui/public/i18n/locales/ko/common.json:
##########
@@ -18,10 +18,14 @@
"asset_other": "에셋들",
"assetEvent_one": "에셋 이벤트",
"assetEvent_other": "에셋 이벤트들",
+ "assetInactive": {
+ "tooltip": "상위 에셋이 비활성화되어 있습니다. 다시 활성화될 때까지 스케줄러가 파티션 평가를 보류합니다."
Review Comment:
```suggestion
"tooltip": "업스트림 에셋이 비활성화되어 있습니다. 다시 활성화될 때까지 스케줄러가 파티션 평가를 보류합니다."
```
To maintain consistency with other keys where `upstream` is translated as
`업스트림` (e.g., `upstream_failed` -> `업스트림 실패`), I suggest changing "상위 에셋" to
"업스트림 에셋" here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
