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

Feature project search fix #43

Merged
merged 9 commits into from
Aug 10, 2023
Merged

Feature project search fix #43

merged 9 commits into from
Aug 10, 2023

Conversation

cmilk18
Copy link
Collaborator

@cmilk18 cmilk18 commented Aug 9, 2023

PR Checklist

아래 요구사항을 만족하는지 체크:

  • 작성한 코드에 대한 테스트 코드 작성 (기능 추가 / 버그 개선)
  • 작성한 코드 문서화 (기능 추가 / 버그 개선)

PR Type

해당 PR이 포함하는 내용 체크:

  • 패키지
    • 패키지 추가
    • 패키지 설정 변경
  • CI
    • CI 구성 추가
    • CI 설정 변경
  • 문서화
    • 신규 문서 추가
    • 기존 문서 변경
  • 신규 기능 추가
  • 버그 수정
  • 코드 개선
    • 성능 개선
    • UI 개선
    • 성능에 영향을 끼치지 않는 수정
  • 테스트
    • 테스트 코드 추가
    • 기존 테스트 코드 변경
  • 커밋 되돌리기
  • 기타

해당 PR에 대한 설명

#42

스크린샷

기타 정보

DTO일부 수정
유저역할을 확인하는 기능 추가
type : fix
body : 검색 기능 버그 수정
type:change
body : 권한확인이 필요한 기능에 권한확인 기능 적용
@cmilk18 cmilk18 added the enhancement New feature or request label Aug 9, 2023
@cmilk18 cmilk18 self-assigned this Aug 9, 2023
Copy link
Contributor

@kimgihean kimgihean left a comment

Choose a reason for hiding this comment

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

고생해써요~~~ ㅎㅎ

@@ -18,7 +18,8 @@
@Api(tags = "NoteBlockController")
@RestController
@RequiredArgsConstructor
@CrossOrigin(origins = "http://localhost:3000", allowedHeaders = "*", allowCredentials = "true")
@
Copy link
Contributor

Choose a reason for hiding this comment

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

컨트롤 + 알트 + L 하면 InteliJ 가 자동정렬해줍니다.
맥은 Command 버튼 꾹 누르면 나오는 단축키 확인해보세요~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

맥은 추가설치 해야하지 않나요?

return role;
}

@Override
Copy link
Contributor

Choose a reason for hiding this comment

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

권한에 따른 페이지 접속 여부는 Spring Security에서 filter로 가능한 것으로 알고있어여~

확인해보시면 좋을 것 같아요!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

??
@cmilk18 cmilk18 closed this Aug 9, 2023
@cmilk18 cmilk18 reopened this Aug 9, 2023
@cmilk18 cmilk18 merged commit 4102d19 into develop Aug 10, 2023
@cmilk18 cmilk18 deleted the feature_project_search_fix branch August 15, 2023 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants