Skip to content
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.

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

[fix] 버그 수정 #270

Merged
merged 9 commits into from
Dec 2, 2024
Merged

[fix] 버그 수정 #270

merged 9 commits into from
Dec 2, 2024

Conversation

easyhak
Copy link
Contributor

@easyhak easyhak commented Dec 2, 2024

closes: #256 #252 #257 #259

🤷‍♂️ Description

  • 새로고침시 좋아요 풀리는 문제 수정
  • 새로고침시 커뮤니티 화면 맨 위로
  • 띄어쓰기

@easyhak easyhak added the 🐞 Bug 애플리케이션에서 발견된 오류나 예상하지 못한 동작을 보고합니다. label Dec 2, 2024
@easyhak easyhak requested a review from a team as a code owner December 2, 2024 07:24
Copy link
Contributor

@ruh0n ruh0n left a comment

Choose a reason for hiding this comment

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

흐음...

.padding(8.dp)
.clickable(onClick = onClick),
.clickable(onClick = onClick)
.padding(8.dp),
Copy link
Contributor

Choose a reason for hiding this comment

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

내부가 아닌 외부에서 하는 게 더 좋을 것 같긴 하지만 괜찮습니다.

)
} else {
postUi
}
Copy link
Contributor

Choose a reason for hiding this comment

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

죽음의 if문 ㄷㄷ 3중첩이라뇨;;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정완

}
} ?: this
}

Copy link
Contributor

Choose a reason for hiding this comment

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

if가 아니게 되었을 뿐이지 똑같잖아요 ㅋㅋㅋㅋㅋ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어쩔수 없습니다ㅠ

Copy link
Contributor

@flashdragon flashdragon left a comment

Choose a reason for hiding this comment

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

@easyhak easyhak merged commit 5b0c12c into dev Dec 2, 2024
2 checks passed
@easyhak easyhak deleted the feature/feedback-1 branch December 2, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug 애플리케이션에서 발견된 오류나 예상하지 못한 동작을 보고합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

테마 설정 / 잠금 설정
3 participants