Skip to content

[fix] 이미지가 정상적으로 불러와지지않는 문제 수정#1104

Merged
seongwon030 merged 1 commit intodevelop-fefrom
fix/#1094-singquote-issue-MOA-555
Jan 25, 2026
Merged

[fix] 이미지가 정상적으로 불러와지지않는 문제 수정#1104
seongwon030 merged 1 commit intodevelop-fefrom
fix/#1094-singquote-issue-MOA-555

Conversation

@lepitaaar
Copy link
Contributor

@lepitaaar lepitaaar commented Jan 25, 2026

#️⃣연관된 이슈

#1094

📝작업 내용

asset을 불러올때 인코딩 문제로 url() 안 css 속성이 정상적으로 인식되지않아 이미지가 표시되지않은 문제가 발생했습니다.
url() 속성에 쌍따옴표를 추가하여 해당문제를 수정했습니다

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

릴리스 노트

  • 스타일
    • 관리자 페이지의 지원자 탭 내 자산 URL 참조 형식을 최적화하여 코드 일관성을 개선했습니다. 사용자에게 보이는 기능 변화는 없습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@lepitaaar lepitaaar self-assigned this Jan 25, 2026
@lepitaaar lepitaaar added 💻 FE Frontend 🛠Fix 기능이 의도한 대로 동작하지 않는 버그를 수정 labels Jan 25, 2026
@vercel
Copy link

vercel bot commented Jan 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
moadong Ready Ready Preview, Comment Jan 25, 2026 6:38am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 25, 2026

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

CSS-in-JS 파일의 url() 함수 내에서 자산 URL 참조 방식을 수정하였습니다. 백그라운드 이미지와 아이콘 URL을 감싸는 따옴표 문법을 표준화하여 자산 불러오기 문제를 해결했습니다. 로직이나 기능적 변화는 없습니다.

Changes

집합 / 파일명 변경 사항
CSS URL 문법 정규화
frontend/src/pages/AdminPage/tabs/ApplicantsTab/ApplicantDetailPage/ApplicantDetailPage.styles.ts, frontend/src/pages/AdminPage/tabs/ApplicantsTab/ApplicantsTab.styles.ts
url() 함수 내 템플릿 리터럴 변수를 명시적 따옴표로 감싸기: url(${...})url("${...}"). 2개 파일에서 총 5개 URL 참조 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • seongwon030
  • oesnuj
  • suhyun113
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 정확히 반영하고 있으며, 이미지 로딩 문제 수정이라는 주요 변경 내용을 명확하게 설명합니다.
Linked Issues check ✅ Passed PR 변경 사항이 연결된 이슈 MOA-555의 요구 사항을 충족합니다. CSS url()에 쌍따옴표를 추가하여 싱글 쿼트로 인한 자산 로딩 문제를 해결했습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 CSS url() 문법 수정과 관련된 범위 내의 변경입니다. 불필요한 추가 변경이나 범위를 벗어난 수정은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@seongwon030 seongwon030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다

@seongwon030 seongwon030 merged commit b71ac8c into develop-fe Jan 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend 🛠Fix 기능이 의도한 대로 동작하지 않는 버그를 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments