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

[BE/#134] GET /posts/block API 구현 #139

Merged
merged 7 commits into from
Nov 23, 2023
Merged

[BE/#134] GET /posts/block API 구현 #139

merged 7 commits into from
Nov 23, 2023

Conversation

koomin1227
Copy link
Member

이슈

체크리스트

  • 유저가 차단한 게시물의 목록을 반환하는 기능 구현
  • 게시글 차단 코드에서 이미 존재하는지 확인하는 코드에 null 이 들어오면 에러 뜨는오류 수정

고민한 내용

null 들어오면 오류 나는 이슈

isExist.status === true 에서 isExist 에서 존재 하지 않으면 null 반환되고 null.status는 null이라고 생각하여 이렇게 구현했었는데 null에 프로퍼티를 찾는 연산을 하면 에러가 떴다.
그래서 null이면 조건문을 통과하도록 수정하였다.

Join 에러

엔티티를 정확하게 작성하지 않아 join이 되지 않는 오류가 발생하였다. 엔티티의 join 관계를 정확히 수정하니 정상적으로 작동하였다.

thumbnail

차단된 게시글의 내용과 게시글의 첫번째 사진을 반환해야했는데 사진을 또 찾으려면 3중 조인을 해야해서 비효율적이라 생각이 들어 게시글을 저장 할 떄 첫번째 사진은 게시글 테이블 안에 저장하도록 수정했다.

스크린샷

@koomin1227 koomin1227 added BE BE 관련 post-hide 게시글 숨김 labels Nov 23, 2023
@koomin1227 koomin1227 self-assigned this Nov 23, 2023
@@ -14,10 +14,10 @@ export class BlockPostEntity {
status: boolean;

@ManyToOne(() => UserEntity, (blocker) => blocker.user_hash)
@JoinColumn({ name: 'blocker' })
@JoinColumn({ name: 'blocker', referencedColumnName: 'user_hash' })
Copy link
Collaborator

Choose a reason for hiding this comment

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

refereceColumn 적용 좋습니다~


@Get()
async postsBlockList() {
const blockerId: string = 'qwe';
Copy link
Collaborator

Choose a reason for hiding this comment

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

추후에 로그인 구현 이후에 이 부분 같이 바꿉시다~

@namewhat99 namewhat99 merged commit beb592a into BE Nov 23, 2023
1 check passed
@namewhat99 namewhat99 deleted the BE-GetPostsBlock-#134 branch November 23, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE BE 관련 post-hide 게시글 숨김
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants