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

Refactor/컬럼명 수정 및 컨버터 추가 #78

Merged
merged 3 commits into from
May 8, 2024

Conversation

This2sho
Copy link
Contributor

🤔세부 내용

  • 데이터베이스에 결제 가능 타입이 description, 위치가 point로 저장되어 있어서 코드 및 flyway 추가했습니다.
  • converter추가하면서 다른 부분들도 추가해줬습니다.

🫵리뷰 중점사항

  • TimeInfo도 DB에 시작 시간, 종료 시간 두개의 컬럼으로 저장되고 있는데 시작 시간 ~ 종료 시간 이렇게 하나의 컬럼으로 통합하는 건 어떻게 생각하시나유?

point -> location
description -> pay_types
@This2sho This2sho added 🪓refactor 기능 수정 🎅🏼deploy 해당 PR 머지시 배포 labels Apr 24, 2024
@This2sho This2sho self-assigned this Apr 24, 2024
Copy link

github-actions bot commented Apr 24, 2024

Test Results

124 tests   124 ✅  8s ⏱️
 25 suites    0 💤
 25 files      0 ❌

Results for commit 2d49f3f.

♻️ This comment has been updated with latest results.

- NO_INFO로 조회시 모든 결제 방식 허용하도록 수정
@This2sho This2sho merged commit baed9ec into main May 8, 2024
3 checks passed
@This2sho This2sho deleted the Refactor/컬럼명_수정_및_컨버터_추가 branch May 8, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎅🏼deploy 해당 PR 머지시 배포 🪓refactor 기능 수정
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant