Feat : classification post list get api, 폴더별 post list get api 수정, 폴더 이름 list get api 에 정렬 적용 #39
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dev-deploy <- feature/ai lambda
- 링크 많은 순 - 폴더 생성순
- AIPostListResponse 수정 - 전체 post list get api, 폴더별 post list get api 페이지 네이션 적용
hye-on
commented
Jul 5, 2024
Comment on lines
+64
to
+83
| @Patch('/posts') | ||
| @PatchAIPostDocs | ||
| async moveAllPost( | ||
| @GetUser() userId: string, | ||
| @Query('suggestionFolderId') suggestionFolderId: string, | ||
| ) { | ||
| await this.classificationService.moveAllPostTosuggestionFolder( | ||
| userId, | ||
| suggestionFolderId, | ||
| ); | ||
| } | ||
|
|
||
| @Delete('/posts/:postId') | ||
| @DeleteAIClassificationDocs | ||
| async abortClassification( | ||
| @GetUser() userId: string, | ||
| @Query('postId') postId: string, | ||
| ) { | ||
| await this.classificationService.abortClassification(userId, postId); | ||
| } |
hye-on
commented
Jul 5, 2024
Comment on lines
+57
to
+64
| }, | ||
| { | ||
| $match: { | ||
| 'aiClassification.deletedAt': null, | ||
| 'aiClassification.suggestedFolderId': suggestedFolderId, | ||
| }, | ||
| }, | ||
| { |
There was a problem hiding this comment.
post에서 ai_classification을 populate해서
'aiClassification.deletedAt': null,
'aiClassification.suggestedFolderId': suggestedFolderId,
인 조건을 만족하는 post{ ai_classification } 을 가져오고 싶었는데
이 조건을 만족하지 않아도 불러와집니다. ( ai_classification이 null인 상태로 post도 가져와짐)
ex ) 내가 원하는 애 - 나중에 읽을 링크에 있으면서 deletedAt이 null이라 분류추천해줘야 하는 애
post {
a,
folderId : '나중에 읽을 링크',
suggestedFolderId {
deletedAt :null
}
}
원하지 않는데 불러지는 애 - (이미, 정치, 아이돌 폴더에 있는 애들도 나옴)
post {
a,
folderId : '정치',
suggestedFolderId : null
}
안에 있는 subdocument에 조건을 걸어도 post에 건 조건이 아니면 필터링이 안됨 ㅠ
그래서 생각한게 아래 두 방법.
- folderId에서 user의
나중에 읽을 링크폴더 id를 가지고 post에 조건 추가해주기- 근데 그래도 똑같은 문제 있음
- { ex) 정치, 아이돌 }에 있는 애들은 안가져올 수 있음
but ,나중에 읽은 링크에 있지만 x 눌러서 더이상 추천에 뜨면 안되는애들까지 불러짐
- { ex) 정치, 아이돌 }에 있는 애들은 안가져올 수 있음
- 근데 그래도 똑같은 문제 있음
- 현재 방법... aggregate
혹시 더 좋은 방법 찾으면 제보 부탁 드립니다...
- populate를 사용해서 서비스 단에서 걸러도 되기는 하는데 페이지네이션땜에 안 되었습니다...
hye-on
commented
Jul 5, 2024
Comment on lines
5
to
19
| export class AIFolderNameListResponse { | ||
| @ApiProperty({ | ||
| type: AIFolderNameServiceDto, | ||
| description: 'ai로 분류된 링크의 총 개수', | ||
| }) | ||
| totalCounts: number; | ||
|
|
||
| @ApiProperty({ | ||
| isArray: true, | ||
| }) | ||
| list: AIFolderNameServiceDto[]; | ||
| list: ClassificationFolderWithCount[]; | ||
|
|
||
| constructor(data: AIFolderNameServiceDto[]) { | ||
| constructor(data: ClassificationFolderWithCount[]) { | ||
| this.totalCounts = data.reduce((sum, folder) => sum + folder.postCount, 0); | ||
| this.list = data; | ||
| } |
There was a problem hiding this comment.
전체 폴더 따로 안 넣고 총 개수 totalCounts로 추가
There was a problem hiding this comment.
ai 분류 페이지에는 전체만 있어서 이렇게 해도 상관없지만
다른 곳은 즐겨찾기랑 나중에 읽을 링크도 있는데
(전자는 폴더도 아이디도 없어서,,)
어떻게 할꺼야?? @Marades
…mash-up-kr/Dorabangs_Node into feature/classification-list-api
|
(/¯◡‿◡)/¯✧·˚ : *✧·˚ :
아무래도 카테고리 분류된 폴더 리스트를 가져오는거다 보니 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 내용
추가 및 변경 사항
참고
PR 중점사항
totalCounts로 전체 개수 다 줬는데 의견 부탁드립니다.스크린샷
폴더별 get api
전체 get api