Skip to content

fix: render rate limit error link as clickable HTML + add web build to CI#7

Merged
let-sunny merged 3 commits intomainfrom
claude/check-duplicate-issues-KOPNN
Mar 24, 2026
Merged

fix: render rate limit error link as clickable HTML + add web build to CI#7
let-sunny merged 3 commits intomainfrom
claude/check-duplicate-issues-KOPNN

Conversation

@let-sunny
Copy link
Copy Markdown
Owner

@let-sunny let-sunny commented Mar 24, 2026

Summary

  • 웹앱에서 429 rate limit 에러 발생 시 링크가 텍스트로만 표시되던 버그 수정
    • showErrortextContent 할당으로 HTML이 이스케이프되던 문제를 insertAdjacentHTML로 별도 삽입
  • CI에 build-web job 추가하여 PR 단계에서 웹 빌드 깨짐을 사전 감지
    • test와 병렬 실행되는 독립 job으로 분리

Changed files

파일 변경
app/web/src/index.html 429 에러 링크를 insertAdjacentHTML로 렌더링
.github/workflows/ci.yml build-web job 추가

Test plan

  • 웹앱에서 Figma API 429 에러 발생 시 "Check your plan limits" 링크가 클릭 가능한지 확인
  • 다른 에러 메시지(403, 404) 정상 동작 확인
  • CI에서 build-web job 정상 실행 확인

https://claude.ai/code/session_0133xaLPFKPrassNBkfV3ihM

@let-sunny let-sunny force-pushed the claude/check-duplicate-issues-KOPNN branch from 378a0ff to 36510c8 Compare March 24, 2026 02:32
claude added 2 commits March 24, 2026 02:33
PR 단계에서 웹 빌드 깨짐을 잡을 수 있도록 build:web 추가.

https://claude.ai/code/session_0133xaLPFKPrassNBkfV3ihM
웹 빌드를 독립 job으로 분리하여 test job과 병렬 실행.

https://claude.ai/code/session_0133xaLPFKPrassNBkfV3ihM
@let-sunny let-sunny changed the title fix: render rate limit error link as clickable HTML fix: render rate limit error link as clickable HTML + add web build to CI Mar 24, 2026
@let-sunny let-sunny merged commit 43f5bdf into main Mar 24, 2026
2 checks passed
@let-sunny let-sunny deleted the claude/check-duplicate-issues-KOPNN branch March 24, 2026 02:41
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.

2 participants