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

골 팀 골, 사용자 fk명 수정 #101

Merged
merged 2 commits into from
Mar 30, 2024
Merged

골 팀 골, 사용자 fk명 수정 #101

merged 2 commits into from
Mar 30, 2024

Conversation

jhsseonn
Copy link
Member

@jhsseonn jhsseonn commented Mar 30, 2024

골 팀에서 골, 사용자의 fk명이 컨벤션과 달리 반대로 되어 있어 수정했습니다.

@jhsseonn jhsseonn added the refactor 코드 리팩터링 label Mar 30, 2024
@jhsseonn jhsseonn requested a review from JJ503 March 30, 2024 07:22
@jhsseonn jhsseonn self-assigned this Mar 30, 2024
Copy link

Copy link
Member

@JJ503 JJ503 left a comment

Choose a reason for hiding this comment

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

[flyway 측 PR 리뷰](https://github.com//pull/95)에서 스탬프의 content도 최대 30글자로 되어 있다고 말씀해 주셨는데, 해당 부분은 제약조건으로 설정할 필요 없을까요? 컬럼 설정으로도 해두면 필드만 보아도 조건을 바로 확인할 수 있고, 실수로 db에 30글자를 초과하는 데이터도 저장되는 것도 방지할 수 있으니 좋지 않을까 하여 여쭤봅니다! 답변 해주시면 바로 approve로 수정하도록 하겠습니다! 사용자의 이름 같은 경우는 아래처럼 설정해 둔 상태입니다. ``` java @column(name = "name", unique = true, length = MAX_LENGTH, nullable = false) private String value; ```

Copy link
Member

@JJ503 JJ503 left a comment

Choose a reason for hiding this comment

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

죄송합니다..!
제가 Stamp가 아닌 StampReport를 보고 리뷰했네요..!
approve로 수정하겠습니다.
해당 pr 리뷰 merge 후 flyway 리뷰 반영하도록 하겠습니다.

@jhsseonn jhsseonn merged commit 0c7856c into develop Mar 30, 2024
2 checks passed
@jhsseonn jhsseonn deleted the refactor/100 branch March 30, 2024 09:12
jhsseonn added a commit that referenced this pull request Apr 3, 2024
* refactor: goalTeam goal, user fk명 수정

* docs: api 문서 업데이트
jhsseonn added a commit that referenced this pull request May 20, 2024
* refacotr: exception 클래스명 리네이밍

* refacotr: exception 클래스명 리네이밍

* feat: 골 참여자 방출 기능 구현

* feat: 골 참여자 방출 컨트롤러 기능 구현

* feat: 방출한 사용자의 모든 스탬프 삭제 기능 구현

* test: 골 팀 삭제했을 경우 삭제 상태가 거짓인지 반환하는 테스트 추가

* docs: api 문서 업데이트

* refactor: 골 참여자 방출 api 형식 변경

* docs: api 문서 업데이트

* docs: api 문서 업데이트

* docs: 논의사항 편집

* refactor: 주석 제거

* refactor: teams 팀원 제거 로직 수정

* refactor: teams 팀원 제거 로직 수정

* refactor: 골 참여자 방출 로직 수정

* refactor: 골 참여자 방출 컨트롤러 로직 수정

* refactor: 골 참여자 방출 메서드명 변경

* refactor: 골 참여자 방출 메서드명 변경

* docs: api 문서 업데이트

* refactor: #100 �골 팀 골, 사용자 fk명 수정 (#101)

* refactor: goalTeam goal, user fk명 수정

* docs: api 문서 업데이트

* ci: #89 Flyway 적용 (#95)

* chore: flyway 의존성 추가 및 설정

* docs: flyway를 위한 테이블 초기 스크립트 추가

* ci: 서브모듈 최신화

* docs: 스탬프 컬럼 최신화 적용

* docs: flyway 스크립트 실패 시 자동 제거 스크립트 추가

* docs: 실패 자동 제거 테스트를 위한 스크립트 추가

* ci: flyway 스크립트 검증 workflow 추가

* fix: 테스트에서 flyway는 수행하지 않도록 수정

* fix: flyway 검증이 안 되는 문제 해결

* ci: 서브모듈 최신화

* ci: 서브모듈 최신화

* ci: 서브모듈 최신화

* docs: 콜백 스크립트 명 수정

* docs: 콜백 스크립트 명 수정

* docs: 콜백 스크립트 명 수정

* docs: 테스트를 위한 flyway 스크립트 제거

* chore: flyway 잘못된 버전 수정

* refactor: 메시지의 컬럼 타입 변경

text -> varchar

* docs: fk 변경사항 수정

fk_goal_team_user <-> fk_goal_team_goal

* docs: api 문서 업데이트

* refactor: 골 참여자 방출시 hardDelete하도록 로직 변경

* refactor: isDeletedFalse 삭제

* refactor: 골 팀 hardDelete 로직 수정

* docs: api 문서 업데이트

* docs: api 문서 업데이트

* refactor: 테스트 필드명 변경

* refactor: 객체 assertSotfly 외부에서 선언

* docs: api 문서 업데이트

---------

Co-authored-by: 제이미 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 코드 리팩터링
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Refact] 골 팀 fk명 수정
2 participants