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:[snsproject] 댓글 목록 UI 구현 및 댓글 데이터 추가 #33

Merged
merged 2 commits into from
May 2, 2024

Conversation

Yoon-Chan
Copy link
Owner

close #18

PR 템플릿

@Yoon-Chan Yoon-Chan added this to the list milestone May 2, 2024
@Yoon-Chan Yoon-Chan self-assigned this May 2, 2024
@@ -21,6 +21,9 @@ data class BoardDTO(
val createUserName: String,
@SerializedName("createUserProfileFilePath")
val createUserProfileFilePath: String,
@SerializedName("commentList")
val commentList: List<CommentDTO>
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-declaration-site reported by reviewdog 🐶
Missing trailing comma before ")"

@@ -21,6 +21,9 @@ data class BoardDTO(
val createUserName: String,
@SerializedName("createUserProfileFilePath")
val createUserProfileFilePath: String,
@SerializedName("commentList")
val commentList: List<CommentDTO>

Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:no-blank-line-in-list reported by reviewdog 🐶
Unexpected blank line(s) in value parameter list

@@ -32,7 +35,8 @@ data class BoardDTO(
profileImageUrl = this.createUserProfileFilePath,
content = contentParam.text,
userId = this.createUserId,
images = contentParam.images
images = contentParam.images,
comments = this.commentList.map { it.toDomainModel() }
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-call-site reported by reviewdog 🐶
Missing trailing comma before ")"

import com.example.domain.model.Comment
import com.google.gson.annotations.SerializedName


Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:no-consecutive-blank-lines reported by reviewdog 🐶
Needless blank line(s)

@SerializedName("createUserName")
val createUserName: String,
@SerializedName("profileImageUrl")
val profileImageUrl: String
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-declaration-site reported by reviewdog 🐶
Missing trailing comma before ")"

val profileImageUrl: String
)

fun CommentDTO.toDomainModel() : Comment {
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:function-return-type-spacing reported by reviewdog 🐶
Unexpected whitespace

val profileImageUrl: String
)

fun CommentDTO.toDomainModel() : Comment {
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:colon-spacing reported by reviewdog 🐶
Unexpected spacing before ":"

id = id,
profileImageUrl = profileImageUrl,
text = comment,
username = createUserName
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-call-site reported by reviewdog 🐶
Missing trailing comma before ")"

@@ -7,5 +7,6 @@ data class Board(
val images: List<String> = listOf(),
val userId: Long,
val username: String,
val profileImageUrl: String
val profileImageUrl: String,
val comments: List<Comment>
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-declaration-site reported by reviewdog 🐶
Missing trailing comma before ")"

val id: Long,
val text: String,
val username: String,
val profileImageUrl: String? = null
Copy link

Choose a reason for hiding this comment

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

🚫 [ktlint] standard:trailing-comma-on-declaration-site reported by reviewdog 🐶
Missing trailing comma before ")"

@Yoon-Chan Yoon-Chan merged commit 7d8581d into main May 2, 2024
4 checks passed
@Yoon-Chan Yoon-Chan deleted the comment/comment_list branch May 2, 2024 16:01
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.

댓글 -댓글 목록 보여주기
1 participant