Skip to content

Fix/monthly chart logic#88

Merged
spde3289 merged 10 commits into
developfrom
fix/monthly-chart-logic
Nov 20, 2025
Merged

Fix/monthly chart logic#88
spde3289 merged 10 commits into
developfrom
fix/monthly-chart-logic

Conversation

@spde3289

@spde3289 spde3289 commented Nov 20, 2025

Copy link
Copy Markdown
Member

개요

카테고리

  • Bug fix (not modifying existing features)
  • New feature (not modifying existing features)
  • Changing feature (modifying existing feature or fixing bug)
  • Docs updated required
  • Chore

상세 내용

이달의 차트 영역 로직 수정

  • 이달의 차트 api 요청 로직 수정
    • 잘못된 경로의 엔드포인트 이용한 구현 -> 올바른 엔드포인트 수정 + 구현 로직 수정

차트 투표하기 모달창 구현 내용 수정

  • 모달창이 열릴 때 컨텐츠 영역에 api요청이 한번 더 이루어지고 있던 상황
    • 현재 보여주고 있는 이달의 차트 데이터를 그대로 모달의 props로 넘겨줘 딜레이 없이 보여지도록 수정
  • 투표하기를 눌렀을 때 크레딧이 차감되고 있지 않던 문제
    • 로컬스토리지에서 직접 차감하고 있던 로직 -> 전역으로 관리하고 있던 크레딧 context를 이용해 차감
  • 투표가 성공적으로 이루어졌을 때 리스트가 최신 정보로 갱신이 되지 않던 문제
    • 투표 성공시 api요청을 한번 더 보내 리스트를 갱신 하도록 수정
  • 모바일 사이즈일 때 스크롤이 안되던 문제 수정
  • 모바일 사이즈일 때 하단의 투표하기 버튼의 레이아웃이 망가지던 상황 수정

@spde3289 spde3289 self-assigned this Nov 20, 2025
@spde3289 spde3289 added the Fix 버그 및 버그 수정 label Nov 20, 2025
@github-project-automation github-project-automation Bot moved this to 코드리뷰 대기 in Fandom-K Nov 20, 2025
@vercel

vercel Bot commented Nov 20, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
fandom-k Ignored Ignored Nov 20, 2025 7:25am

@spde3289 spde3289 merged commit 64f16f4 into develop Nov 20, 2025
3 checks passed
@spde3289 spde3289 deleted the fix/monthly-chart-logic branch November 20, 2025 07:25
@github-project-automation github-project-automation Bot moved this from 코드리뷰 대기 to 완료 in Fandom-K Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix 버그 및 버그 수정

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

1 participant