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

리뷰용 코드업로드 #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

리뷰용 코드업로드 #10

wants to merge 1 commit into from

Conversation

Dothur
Copy link
Collaborator

@Dothur Dothur commented Aug 8, 2023

No description provided.

}

// 현재 로그인한 사용자 정보 가져오기
public UserEntity getUser() {
Copy link
Contributor

Choose a reason for hiding this comment

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

권한 토큰을 가져오는 부분이 인상깊었습니다 !

Copy link

Choose a reason for hiding this comment

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

댓글 기능 구현 열심히 생각하신게 보이네요!! 코드 참고하겠습니다!

Copy link
Contributor

Choose a reason for hiding this comment

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

리드미를 정말 깔끔하게 잘 작성하셨네요 ! 참고하고 갑니다 !!!


#### 2. **_deleted_at 수정_**

#### 3. **_Postman Collection 업로드 & Readme update & 설명 주석 추가_**
Copy link

Choose a reason for hiding this comment

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

리드미는 이렇게 작성해야되는거군요...너무 깔끔해서 보기도 좋고 한눈에 들어옵니다
센스가 부럽네요

String[] fileNameSplit = originalFilename.split("\\.");
String extension = fileNameSplit[fileNameSplit.length - 1];
String profileFilename = "image." + extension;
String profilePath = profileDir + profileFilename;
Copy link
Contributor

Choose a reason for hiding this comment

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

오 이런식으로 하면 되는군요 ! 조언해주신 부분 참고해서 다시 구현해보겠습니다 감사합니다 !!

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.

5 participants