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

(batch) be KAN-80: 좋아요 기능을 위한 DB 생성 코드 업데이트 #19

Closed
wants to merge 7 commits into from

Conversation

one0955
Copy link
Contributor

@one0955 one0955 commented May 21, 2024

KAN-80 좋아요 기능 테스트를 위해 DB 스키마 변경을 진행하였습니다.

python main.py 를 실행하기만 하면 기존 테이블 삭제 후 다시 테이블을 생성합니다.

Copy link
Member

@sinkyoungdeok sinkyoungdeok left a comment

Choose a reason for hiding this comment

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

여긴 스키마를 위한 코드를 관리하지 않습니다.
혹시 be 쪽 코드 실행안하면서 개발하고있나요?

@one0955
Copy link
Contributor Author

one0955 commented May 22, 2024

여긴 스키마를 위한 코드를 관리하지 않습니다. 혹시 be 쪽 코드 실행안하면서 개발하고있나요?

테이블 스키마 main pull 땡겨온거에 맞췄습니다. 열이름이랑 타입이 미세하게 다르더라고요. (즉, 땡겨온 엔티티에 맞게 수정)

@sinkyoungdeok
Copy link
Member

@one0955 main pull 안땡겨와진걸로 보이는데, 확인한번 해보시겠어여? PR filechanges 확인해보세용

@one0955
Copy link
Contributor Author

one0955 commented May 22, 2024

어제 반영된 변경사항도 업데이트해서 다시 올립니다

Copy link
Member

@sinkyoungdeok sinkyoungdeok left a comment

Choose a reason for hiding this comment

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

file changed 쪽 검토 한번 부탁드립니다.
제가 작업한거 거의 날리고 작업하신 것으로 보이는데, 혹시 깃이 아직 익숙하지 않으신건가요?

@one0955
Copy link
Contributor Author

one0955 commented May 23, 2024

아 mysql_batch 폴더 코드 수정 부분이 있었네요. ㅎ 저만 이 폴더 작업하는줄 알았는데.. 브랜치 하나 따로 만들어서 다시 올리겠습니다.

@sinkyoungdeok
Copy link
Member

@one0955 넵 ㅎㅎ 리뷰받기전에 file changes 쪽도 한번씩 검토해주세요 ㅠㅠ

@sinkyoungdeok sinkyoungdeok deleted the KAN-81 branch May 26, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants