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

[Fix] Domain 수정 #11

Merged
merged 3 commits into from
Dec 17, 2024
Merged

[Fix] Domain 수정 #11

merged 3 commits into from
Dec 17, 2024

Conversation

KyunghwanChoi
Copy link
Collaborator

Related issue 🛠

Work Description ✏️

각 엔티티에서 누락된 애너테이션 추가, ERD 변동사항 반영

Trouble Shooting ⚽️

Related ScreenShot 📷

Uncompleted Tasks 😅

To Reviewers 📢

@KyunghwanChoi KyunghwanChoi self-assigned this Dec 17, 2024
@KyunghwanChoi KyunghwanChoi linked an issue Dec 17, 2024 that may be closed by this pull request
Copy link
Owner

@seoshinehyo seoshinehyo left a comment

Choose a reason for hiding this comment

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

develop 브랜치의 구조에 맞게 코드를 작성 안 하셔서 코드가 다 꼬였어요. develop 브랜치 구조에 맞는 디렉터리에 코드 작성해 주셔야 될 것 같습니다!

Copy link
Owner

@seoshinehyo seoshinehyo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
코멘트 남겼으니 확인 부탁드려요!

@@ -35,3 +35,4 @@ out/

### VS Code ###
.vscode/
.DS_Store
Copy link
Owner

Choose a reason for hiding this comment

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

혹시 .DS_Store은 무슨 용도의 파일인건가요??

import org.springframework.context.annotation.Configuration;

@Configuration
public class SwaggerConfig {
Copy link
Owner

Choose a reason for hiding this comment

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

swagger 사이트 접속되게 SwaggerConfig 코드 추가해주세요!

@seoshinehyo seoshinehyo merged commit a7dce69 into develop Dec 17, 2024
1 check passed
jjun522 pushed a commit that referenced this pull request Dec 19, 2024
KyunghwanChoi pushed a commit that referenced this pull request Dec 20, 2024
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.

[Bug or Fix] [Domain 에너테이션 수정]
2 participants