Skip to content

[Feat] 코스 모드 사진 마일스톤을 정상까지 거리로 4등분#197

Open
JangInho wants to merge 1 commit into
developfrom
feat/#196-milestone-up-to-summit
Open

[Feat] 코스 모드 사진 마일스톤을 정상까지 거리로 4등분#197
JangInho wants to merge 1 commit into
developfrom
feat/#196-milestone-up-to-summit

Conversation

@JangInho

@JangInho JangInho commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

🧾 요약

  • 코스 마일스톤(1/4·2/4·3/4·4/4) 을 코스 전체 distance 가 아닌 정상까지(=course.distance/2, 임시 정책) 거리에 균등 분배하도록 변경.

🔗 이슈

✨ 변경 내용

  • 코스 4/4 가 곧 정상 도달 시점이 되어 SUMMIT 임계와 일치한다.
  • TrackingMilestoneCalculator.courseMilestones: summitMeters = totalMeters/2 도입
  • TrackingMilestoneTriggerService.evaluateSummit: 호출자가 정상 지점을 그대로 전달하므로 내부 halfwayMark/2 제거.
  • 변수/파라미터/payload 키를 summitMark 로 통일
  • 자유 기록(500m 6컷) / NotificationType / FCM 흐름 / DB 영향 없음

✅ 확인

  • 빌드 OK
  • 테스트 OK

코스 마일스톤(1/4·2/4·3/4·4/4) 을 코스 전체 distance 가 아닌
정상까지(=course.distance/2, 임시 정책) 거리에 균등 분배하도록 변경.
4/4 가 곧 정상 도달 시점이 되어 SUMMIT 임계와 일치한다.

- TrackingMilestoneCalculator.courseMilestones: summitMeters = totalMeters/2 도입
- TrackingMilestoneTriggerService.evaluateSummit: 호출자가 정상 지점을 그대로
  전달하므로 내부 halfwayMark/2 제거. 변수/파라미터/payload 키를 summitMark 로 통일
- 자유 기록(500m 6컷) / NotificationType / FCM 흐름 / DB 영향 없음
@JangInho JangInho self-assigned this Jun 4, 2026
@JangInho JangInho added the enhancement New feature or request label Jun 4, 2026
@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b283c5a7-4bf1-4e72-aa08-82bb191dcc12

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#196-milestone-up-to-summit

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gemini-code-assist

Copy link
Copy Markdown

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

본 PR은 코스 모드에서의 사진 촬영 마일스톤 및 정상 도달 알림 로직을 개선합니다. 기존에 전체 코스 거리를 기준으로 계산되던 마일스톤을 정상 지점(전체 거리의 50%) 기준으로 재설정하여, 4번째 마일스톤이 정상 도달 시점과 일치하도록 조정했습니다. 이를 통해 알림 로직의 일관성을 확보하고 향후 정상 좌표 변경에 유연하게 대응할 수 있도록 구조를 개선했습니다.

Highlights

  • 코스 마일스톤 계산 로직 변경: 코스 모드에서 사진 촬영 마일스톤(1/4~4/4)을 전체 거리가 아닌 정상까지의 거리(전체 거리의 50%)를 기준으로 균등 분배하도록 수정했습니다.
  • 정상 도달 알림 로직 개선: 정상 도달 시점 계산을 호출부에서 결정하도록 변경하고, 내부 변수 및 파라미터명을 'summitMark'로 통일하여 코드 가독성과 유지보수성을 높였습니다.
New Features

🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

이번 풀리퀘스트는 코스 따라가기 모드에서 정상 도달 기준을 코스 전체 거리의 절반으로 정의하는 임시 정책을 반영하기 위해 마일스톤 계산 및 정상 도달 알림 평가 로직을 수정합니다. TrackingMilestoneCalculator에서 마일스톤 거리를 정상 기준 거리(summitMeters)에 맞춰 균등 분배하도록 변경하였고, TrackingMilestoneTriggerService에서는 호출부로부터 전달받은 마지막 마일스톤 거리를 정상 지점(summitMark)으로 직접 사용하여 알림을 평가하도록 개선하였습니다. 리뷰 의견이 없으므로 추가적인 피드백은 제공하지 않습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@howooyeon howooyeon self-requested a review June 4, 2026 13:48

@howooyeon howooyeon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good

@howooyeon howooyeon changed the title feat: 코스 모드 사진 마일스톤을 정상까지 거리로 4등분 [Feat] 코스 모드 사진 마일스톤을 정상까지 거리로 4등분 Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 코스에서 4번이 아니라 정상까지 4번 푸시오게 수정

2 participants