Skip to content

[ test ] : #434 무중단 배포 테스트 #438

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
treejh merged 2 commits into from
May 1, 2025
Merged

[ test ] : #434 무중단 배포 테스트 #438

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
09a33c0
[ front / feat ] : #434 팝업창 바로가기 수정
treejh May 1, 2025
dd75200
[ front / feat ] : #434 팝업창 바로가기 수정
treejh May 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion frontend/src/app/post/[postId]/detail/get/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1274,7 +1274,7 @@ export default function DetailPage() {
</div>
<div className="flex-1 min-w-0">
<button
onClick={() => handleProfileClick(post.author)}
onClick={() => handleBlogMainClick(post.username)}
Copy link

@github-actions github-actions bot May 1, 2025

Choose a reason for hiding this comment

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

handleProfileClick 함수 대신 handleBlogMainClick 함수를 사용하는 것으로 변경되었는데, post.author 대신 post.username을 사용하는 이유에 대한 명확한 설명이 필요합니다. 두 변수의 의미 차이와 이 변경이 의도하는 바를 코드 주석이나 commit message에 명확히 기재해야 합니다. 만약 author 필드가 더 이상 사용되지 않는다면, post 객체에서 해당 필드를 제거하는 것을 고려해볼 수 있습니다. 또한, handleBlogMainClick 함수의 구현 내용도 검토하여 예상치 못한 동작이 없는지 확인해야 합니다. handleProfileClick 함수와의 차이점을 명확히 이해하고 변경 사항을 검토해야 합니다.

className="font-medium hover:text-[#2E804E] transition-colors duration-200 truncate block"
>
{post.username}
Expand Down
Loading