fix(cost-explorer): anchor cost summary to calendar month instead of … - #6149
Merged
Conversation
…array position 홈 Cost Summary 위젯이 값은 배열 위치(chartData[len-1])로 집고 날짜 라벨은 dayjs()로 따로 계산해서, 데이터가 끝에서 끊긴 워크스페이스에서 과거 달의 금액이 "이번 달"로 표시됐다. AGGREGATING 뱃지도 하드코딩이라 확정된 데이터에 붙었다. 값과 라벨이 모두 같은 소스(날짜 키)에서 나오도록 바꾸고, 리포트 화면의 기준점도 "마지막 발행 리포트"에서 달력 기준(저번 달)으로 통일한다. - add getLatestMonth()/getCurrentMonth() helper shared by home and cost report - CostSummary: 위치 인덱스 → 날짜 키 조회, is_confirmed 그룹을 월 단위로 병합 (병합 없이 조회하면 중복 중 하나만 집어가 값이 누락됨) - CostSummary: 값이 없으면 '-' 와 사유 문구 표기, 뱃지 숨김 - CostSummaryChart: x축을 period 기준 dense 배열로 채워 indexOf 를 항상 유효하게 하고, 단색 구간에서 visualMap 을 걸지 않도록 수정 (한쪽이 열린 구간만 남으면 echarts 가 gradient stop 을 만들지 못해 라인 렌더가 통째로 깨졌음) - CostSummaryChart: 값 없는 달 툴팁에 '-' 표기 - CostTrendCard/MonthlyTotalAmountSummaryCard: recentReportMonth 앵커 제거 - CostTrendCard: 요약 금액을 period.end 기준으로 조회 (연도 선택 시 라벨과 값이 어긋나 $0 이 표시되던 문제 동반 수정) - MonthlyTotalAmountSummaryCard: currentDate 초기값을 즉시 세팅해 watcher 순서 의존 제거, cost_report_config_id watch 의 이중 화살표 오타 수정 - store: 기준점 역할이 사라진 recentReportMonth 와 미사용 recentIssueDate 제거 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Signed-off-by: yuda <yuda@megazone.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Contributor
|
🎉 @seungyeoneeee has been randomly selected as the reviewer! Please review. 🙏 |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
…array position
홈 Cost Summary 위젯이 값은 배열 위치(chartData[len-1])로 집고 날짜 라벨은 dayjs()로 따로 계산해서, 데이터가 끝에서 끊긴 워크스페이스에서 과거 달의 금액이
"이번 달"로 표시됐다. AGGREGATING 뱃지도 하드코딩이라 확정된 데이터에 붙었다.
값과 라벨이 모두 같은 소스(날짜 키)에서 나오도록 바꾸고, 리포트 화면의 기준점도
"마지막 발행 리포트"에서 달력 기준(저번 달)으로 통일한다.
Skip Review (optional)
style,chore,ci,test,docs)Description (optional)
Things to Talk About (optional)