Skip to content

[Refactor/runimo domain rename] Runimo 관련 도메인 이름 변경#30

Merged
jeeheaG merged 8 commits intomainfrom
refactor/runimo-domain-rename
Apr 7, 2025
Merged

[Refactor/runimo domain rename] Runimo 관련 도메인 이름 변경#30
jeeheaG merged 8 commits intomainfrom
refactor/runimo-domain-rename

Conversation

@jeeheaG
Copy link
Copy Markdown
Contributor

@jeeheaG jeeheaG commented Apr 7, 2025

[작업 내용]

  • 기존 UserRunimo / Runimo 로 사용하던 도메인 이름을
    Runimo / RunimoDefinition 으로 각각 변경
  • 수정된 엔티티 의미에 맞게 일부 로직 수정
    userId-runimoId 복합키를 사용했던 부분들을 Runimo 의 id 를 사용하도록 변경
  • 새로운 Runimo 엔티티에 수치데이터 두 개 컬럼 추가
    • 해당 러니모와 함께 달린 누적 횟수
    • 해당 러니모와 함께 달린 누적 거리
  • 러니모 정적 더미데이터 insert 문 추가 (schema.sql)
  • 부화 api : 러니모 중복 시에도 저장되던 버그 수정

[전달 사항]

변경된 엔티티의 키값 사용 규칙

  • Runimo 는 id, userId, runimoDefinitionId 를 갖는다.
    그 중 id만 runimo_id 라는 이름으로 외부에 노출한다.
  • RunimoDefinition 의 id 는 외부에 노출시키지 않는다. code 만 노출한다. (e.g. R-101)
    -> runimo_id 와 혼동하지 않기 위함입니다.

[남은 일]

  • 부화 로직 개발
  • 부화 중복 테스트케이스 추가
  • 보유 러니모 조회 api : 응답형식 프론트와 맞추기
  • 러니모 수치데이터 update로직 및 main view 응답 수정

바꾸고 나니 로직 보기가 편안하네요 허허 잘바꿨다

@jeeheaG jeeheaG requested a review from ekgns33 April 7, 2025 08:41
@jeeheaG jeeheaG self-assigned this Apr 7, 2025
Copy link
Copy Markdown
Contributor

@ekgns33 ekgns33 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)

Comment thread src/main/java/org/runimo/runimo/hatch/service/HatchClient.java
@jeeheaG
Copy link
Copy Markdown
Contributor Author

jeeheaG commented Apr 7, 2025

https://github.com/Run-Us/Runimo/actions/runs/14304733508/job/40085987905

@ekgns33 이거 main view 쪽 테스트 터진 거 같은데 맞나요?
괜찮으신지 함 봐주시고 머지 부탁드릴게요~~

@jeeheaG jeeheaG merged commit 20c3ee9 into main Apr 7, 2025
5 of 6 checks passed
@jeeheaG jeeheaG deleted the refactor/runimo-domain-rename branch April 7, 2025 10:35
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.

2 participants