Skip to content

[#95] refactor : 사용하지 않는 Import문, 주석 제거#100

Merged
lswoo0705 merged 1 commit intodevelopfrom
refactor/#95-refactoring
Oct 23, 2024
Merged

[#95] refactor : 사용하지 않는 Import문, 주석 제거#100
lswoo0705 merged 1 commit intodevelopfrom
refactor/#95-refactoring

Conversation

@lswoo0705
Copy link
Copy Markdown
Member

#️⃣연관된 이슈

📝작업 내용

  • 사용하지 않는 Import문, 주석 제거

@lswoo0705 lswoo0705 added the refactor 기능 변경이 없는 코드 리팩토링에 관한 커밋 label Oct 23, 2024
@lswoo0705 lswoo0705 self-assigned this Oct 23, 2024
Copy link
Copy Markdown
Member

@joo-chang joo-chang left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Comment on lines +53 to +55
public CommonResponse<GetStageRes> updateStageById(@PathVariable("stageId") Long stageId,
@Valid @RequestBody UpdateStageReq updateStageReq) {
stageService.updateStageById(stageId, updateStageReq);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

이건 혹시 뭘까요 엔터가 들어간건가..

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

마씁니다
이건 나중에 일괄적으로 통일시켜야할거같습니다

@PreAuthorize("hasRole('MANAGER')")
@DeleteMapping("/{stageId}")
public CommonResponse deleteStageById(@UseAuth CustomUser customUser ,@PathVariable("stageId") Long stageId) {
public CommonResponse deleteStageById(@UseAuth CustomUser customUser, @PathVariable("stageId") Long stageId) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

CommonResponse 로 변경해야되는데 제가 수정했습니다!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

캄솨합니다

@lswoo0705 lswoo0705 merged commit 62467d9 into develop Oct 23, 2024
@lswoo0705 lswoo0705 deleted the refactor/#95-refactoring branch October 23, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 기능 변경이 없는 코드 리팩토링에 관한 커밋

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants