Skip to content

[Hotfix] Fix redundant optional label text in MySQL/MongoDB templates#10

Merged
yessjun merged 2 commits intomainfrom
hotfix/template-optional-fields-rendering
Nov 15, 2025
Merged

[Hotfix] Fix redundant optional label text in MySQL/MongoDB templates#10
yessjun merged 2 commits intomainfrom
hotfix/template-optional-fields-rendering

Conversation

@yessjun
Copy link
Contributor

@yessjun yessjun commented Nov 15, 2025

Fix redundant optional label text in MySQL/MongoDB templates

Summary

  • MySQL/MongoDB 템플릿에서 optional 필드 라벨의 "(선택사항)" 텍스트 제거
  • 15개 전체 템플릿 감사 및 JSON 구조 일관성 검증

Changes

MySQL Template (templates/mysql/config.json)

  • app_username label: "애플리케이션 사용자명 (선택사항)""애플리케이션 사용자명"
  • app_password label: "애플리케이션 비밀번호 (선택사항)""애플리케이션 비밀번호"

MongoDB Template (templates/mongodb/config.json)

  • app_username label: "애플리케이션 사용자명 (선택사항)""애플리케이션 사용자명"
  • app_password label: "애플리케이션 비밀번호 (선택사항)""애플리케이션 비밀번호"

Tekton template engine uses missingkey=error mode, which throws error
when accessing map keys that don't exist. Frontend was not initializing
optional fields with empty default values, causing build failures.

Changes:
- Add "default": "" to app_username in MySQL config
- Add "default": "" to app_username in MongoDB config
- Ensures consistent JSON structure for all optional fields

This prevents "map has no entry for key" errors during Dockerfile
template rendering in Tekton build pipeline.

LP-487
Removes "(선택사항)" text from labels where category is already "optional"
to avoid redundancy in UI. Also ensures consistent JSON structure by
adding missing "default": "" keys.

Changes:
- MySQL: Remove "(선택사항)" from app_username and app_password labels
- MongoDB: Remove "(선택사항)" from app_username and app_password labels
- All templates audited: 13 other templates already correct

This improves UI clarity and maintains consistent JSON structure across
all 15 templates.
@yessjun yessjun merged commit 653f8c1 into main Nov 15, 2025
1 check passed
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.

1 participant