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

feat: 피드 상세보기 뷰 댓글 기능 구현 #256

Merged
merged 18 commits into from
Aug 28, 2024
Merged

feat: 피드 상세보기 뷰 댓글 기능 구현 #256

merged 18 commits into from
Aug 28, 2024

Conversation

s9hn
Copy link
Member

@s9hn s9hn commented Aug 28, 2024

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • 댓글 작성(작성 시 자동 스크롤)
  • 댓글 수정(수정됨 반영)
  • 댓글 삭제
  • 타인 댓글 부적절한 표현 신고
  • 타인 댓글 스포일러 신고

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

36533535.webm

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

@junseo511 junseo511 added the ❌[DO NOT MERGE] 머지를 하면 안됩니다. label Aug 28, 2024
@s9hn s9hn added 🍯 [FEAT] 새로운 기능을 개발합니다. ⚖️ 재상 세훈 전지적 세훈 시점 and removed ❌[DO NOT MERGE] 머지를 하면 안됩니다. labels Aug 28, 2024
@s9hn s9hn changed the title Feat/249 feat: 피드 상세보기 뷰 댓글 기능 구현 Aug 28, 2024
Copy link
Contributor

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

굿티비 잘하셨겠죠

}

suspend fun saveSpoilerFeed(feedId: Long) {
feedApi.postSpoilerFeed(feedId).also { _cachedFeeds.removeIf { it.id == feedId } }
Copy link
Contributor

Choose a reason for hiding this comment

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

a: 오 ~

popupBinding.root,
WindowManager.LayoutParams.WRAP_CONTENT,
WindowManager.LayoutParams.WRAP_CONTENT,
true
Copy link
Contributor

Choose a reason for hiding this comment

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

r: 트레일링 콤마

@@ -1,5 +1,6 @@
package com.teamwss.websoso.ui.feedDetail

import android.util.Log
Copy link
Contributor

Choose a reason for hiding this comment

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

r: 제거

Copy link
Contributor

@librarywon librarywon left a comment

Choose a reason for hiding this comment

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

리뷰 반영만 해주세요! 고생했습니다

@@ -1,12 +1,14 @@
package com.teamwss.websoso.data.repository

import android.util.Log
Copy link
Contributor

Choose a reason for hiding this comment

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

r: 로그 임포트 지워주세요

@@ -47,8 +161,29 @@ class FeedDetailActivity :
private fun updateView(feedDetailUiState: Success) {
val header = Header(feedDetailUiState.feedDetail.feed)
val comments = feedDetailUiState.feedDetail.comments.map { Comment(it) }
val feedDetail = listOf(header) + comments

Copy link
Contributor

Choose a reason for hiding this comment

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

r: 공백

@s9hn s9hn merged commit 015a71c into develop Aug 28, 2024
@librarywon librarywon deleted the feat/249 branch September 11, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚖️ 재상 세훈 전지적 세훈 시점 🍯 [FEAT] 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 피드 상세보기 뷰 댓글 기능 구현
4 participants