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: 조회수 처리, 아이템 수정 불러오기 변경 #92

Merged
merged 15 commits into from
Oct 5, 2023

Conversation

HyowonSin
Copy link

@HyowonSin HyowonSin commented Oct 4, 2023

What is this PR? 👓

Key changes 🔑

  • 아이템 상세 조회 시에 조회수 증가
    → 자기 자신은 증가하지 않음
    → 조회수 증가 시에, 일정 시간 동안 증가하지 않음 (쿠키 활용)
    → 아이템 상세 조회 시에 조회수 증가 반영된 값 출력 (자기 자신의 글이 아니면 조회수 0을 볼 수 없음)

  • 아이템 수정 불러오기 요구사항 반영
    → locations 의 길이는 1~3

  • gpt 모델 4로 변경

To reviewers 👋

  • 조회수 증가 쿼리(업데이트)와 쿠키 처리하는 부분에만 트랜잭션을 붙여서 범위를 좁혀봤음
  • gpt 모델 4가 더 추천을 잘 해주지만 속도가 많이 아쉬워서 고민

@HyowonSin HyowonSin temporarily deployed to dev October 4, 2023 06:02 — with GitHub Actions Inactive
@HyowonSin HyowonSin temporarily deployed to dev October 4, 2023 06:04 — with GitHub Actions Inactive
@HyowonSin HyowonSin temporarily deployed to dev October 4, 2023 06:05 — with GitHub Actions Inactive
@HyowonSin HyowonSin temporarily deployed to dev October 4, 2023 13:15 — with GitHub Actions Inactive
@HyowonSin HyowonSin temporarily deployed to dev October 4, 2023 13:16 — with GitHub Actions Inactive
@sudago sudago merged commit acd39de into be Oct 5, 2023
@HyowonSin HyowonSin deleted the feat/view-count branch October 5, 2023 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants