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

[Feature/store like get] 마이페이지-저장목록 조회 및 가게저장 API 개발 #69

Merged
merged 7 commits into from
Nov 29, 2023

Conversation

dldmsql
Copy link
Member

@dldmsql dldmsql commented Nov 26, 2023

작업 내용

마이페이지-저장목록 조회 및 가게저장 API 개발

관련 이슈

#68

작업 확인 방법

{
  "localDateTime": "2023-11-26T18:43:14.898042",
  "message": "OK",
  "data": {
    "content": [
      {
        "idx": 386,
        "name": "히포 브런치하우스",
        "address": "서울 마포구 연남동 487-34",
        "phoneNumber": "010-3796-3176",
        "categoryDetail": "카페",
        "distance": 1721,
        "grade": 0,
        "reviewCount": 0,
        "recommendedCount": 0,
        "images": null,
        "isLiked": true, // 저장인 경우
        "userLikesCount": 1 // 저장 카운트
      }
    ],
    "pageable": {
      "sort": {
        "empty": true,
        "sorted": false,
        "unsorted": true
      },
      "offset": 0,
      "pageNumber": 0,
      "pageSize": 10,
      "paged": true,
      "unpaged": false
    },
    "last": true,
    "totalPages": 1,
    "totalElements": 1,
    "first": true,
    "size": 10,
    "number": 0,
    "sort": {
      "empty": true,
      "sorted": false,
      "unsorted": true
    },
    "numberOfElements": 1,
    "empty": false
  }
}

추가 정보 (선택 사항)

StoreGetRes 레코드를 사용하려고 했으나 피그마 UI 상 userLikesCount가 추가로 필요해서 record를 새롭게 정의했습니다.

🙋🏻‍♀️질문 !!!

가게를 저장하고 저장 해제하는 것을 하나의 API에서 처리하도록 구현했는데, 혹시 분리되어야 한다고 생각하시나요??

@dldmsql dldmsql added the Feature 기능 개발 label Nov 26, 2023
@dldmsql dldmsql self-assigned this Nov 26, 2023
Copy link
Member

@Qbeom0925 Qbeom0925 left a comment

Choose a reason for hiding this comment

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

질문에 대한 제 답변은
제가 생각했을때는 API는 요청 리소스를 기반으로 파악을하여 작동을 해야하는데 내부의 리소스를 통해 분기처리가 되는 것이 REST 규칙에 어긋난다고 생각합니다!

은비님은 해당 부분에 대해서는 어떻게 생각하시나요?!?

@dldmsql
Copy link
Member Author

dldmsql commented Nov 27, 2023

질문에 대한 제 답변은 제가 생각했을때는 API는 요청 리소스를 기반으로 파악을하여 작동을 해야하는데 내부의 리소스를 통해 분기처리가 되는 것이 REST 규칙에 어긋난다고 생각합니다!

은비님은 해당 부분에 대해서는 어떻게 생각하시나요?!?

저는 운영을 생각했을 때, 해당 로직이 더 확장되거나 변경이 잦아질 구간이 아니고 유지보수 측면에서도 api 1개인 편이 좋을 것 같다고 생각해요. rest 원칙에 어긋나기는 하지만, 완벽한 REST API는 구현하기 힘들다고 생각해서 이 부분은 분기 처리로 하는 편이 좋지 않을까 합니다. ( + 클라에서도 응답으로 t/f를 통해 ui 처리하기 수월할 거라고도 생각했습니다. )

- 저장 시, true
- 저장 해제 시, false
Copy link

sonarcloud bot commented Nov 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@Qbeom0925 Qbeom0925 left a comment

Choose a reason for hiding this comment

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

저는 운영을 생각했을 때, 해당 로직이 더 확장되거나 변경이 잦아질 구간이 아니고 유지보수 측면에서도 api 1개인 편이 좋을 것 같다고 생각해요. rest 원칙에 어긋나기는 하지만, 완벽한 REST API는 구현하기 힘들다고 생각해서 이 부분은 분기 처리로 하는 편이 좋지 않을까 합니다. ( + 클라에서도 응답으로 t/f를 통해 ui 처리하기 수월할 거라고도 생각했습니다. )

은비님 의견에 동의합니다. rest원칙을 지키면서 하기에는 이를 운영 입장에서 본다면 동의합니다!

@Qbeom0925 Qbeom0925 merged commit cec2a66 into develop Nov 29, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants