Closed lcc3108 closed 4 months ago
이번 변경 사항 요약: Alert
컴포넌트를 사용하여 @grafana/ui
에서 불필요한 그래프나 스타일을 제거하고 JSX 구조를 개선했습니다. ConfigEditor.tsx
와 JWTConfig.tsx
파일의 조건부 렌더링과 오류 표시를 재구성하여 코드 가독성을 높였습니다.
파일 경로 | 변경 요약 |
---|---|
src/ConfigEditor.tsx |
Alert 컴포넌트 추가 및 조건부 렌더링 재구성, 정보 표시 관련 JSX 구조 업데이트, 코드 포맷 개선 |
src/JWTConfig.tsx |
오류 표시를 Alert 컴포넌트로 변경, 불필요한 InlineFormLabel 제거 |
Objective (이슈 번호) | Addressed | Explanation |
---|---|---|
grafana-info-box 클래스 제거 (#112) |
✅ | |
alert 및 alert-error 클래스 제거 (#112) |
✅ |
작은 코딩 변화 속에,
그래프가 휘날리며 날아가네.
알림이 더해져,
오류도 친절히 알려주네.
코드는 깔끔하고,
마음은 평온해.
🐰✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
7~8버전의 e2e 테스트는 지속적으로 실패하기때문에 추후 지원버전에서 제거해서 관리할 예정
Summary
fix https://github.com/blackcowmoo/grafana-google-analytics-datasource/issues/112 Replacing scss expiring in grafana version 12 with grafana/ui components
Summary by CodeRabbit