GTB-31 [feat] 입장 완료 구현 #27

yechan-kim · 2024-05-23T12:39:25Z

1. 무슨 이유로 코드를 변경했나요?

사용자가 입장을 완료하면 관리자가 입장 완료 버튼을 눌러 입장을 완료 시키는 API를 개발했습니다.

2. 어떤 위험이나 장애를 발견했나요?

DB를 강제로 주입하면서 UUID가 제대로 생성되지 않는 오류가 있었습니다. 이는 SQL문으로 ID를 직접 삽입해서 생기는 문제로 데이터를 DB IDE에서 직접 삽입해서 문제를 해결하였습니다.
2.QueingCount가 0일 경우에도 완료 로직이 수행되는 것을 수정하였습니다.

3. 관련 스크린샷을 첨부해주세요.

아래 처럼 QueingCount가 0이먄 예외가 발생하도록 처리하였습니다.

4. 완료 사항

입장 완료 기능 구현

5. 추가 사항

rootTiket

의견을 남겨놓았습니다. 읽고 수정혹은 다른 의견 부탁드립니다

src/main/java/site/gachontable/gachontablebe/domain/admin/usecase/ForceCancel.java

rootTiket · 2024-05-24T09:01:48Z

src/main/java/site/gachontable/gachontablebe/global/config/SecurityConfig.java

@@ -50,6 +50,7 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                        authorize
                                .requestMatchers("/v3/api-docs/**", "/swagger-ui/**").permitAll()
                                .requestMatchers("/admin/login").permitAll()
+                                .requestMatchers("/admin/force-cancel").permitAll()


테스트를 위해 권한을 풀어주신 것 같은데 어드민 로그인이 구현되면 꼭 수정해주셔야 할 것 같습니다

rootTiket

의견을 남겨놓았습니다. 읽고 수정혹은 다른 의견 부탁드립니다

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

rootTiket · 2024-05-24T09:15:45Z

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

@@ -35,4 +38,10 @@ public ResponseEntity<JwtResponse> login(@RequestBody AdminLoginRequest request)
        JwtResponse tokens = adminLogin.execute(request.id(), request.password());
        return ResponseEntity.ok(tokens);
    }
+
+    @PostMapping("/force-cancel")
+    public ResponseEntity<String> forceCancel(@RequestBody ForceCancelRequest request) {


현재 토큰이 구현되어 있지 않아 빠져있지만 차후 Authentication 혹은 token 을 받아와야 할 것 같습니다.

Admin의 경우 현재 토큰이 구현되어 있기 때문에 토큰 방식으로 인증 진행하면 될 것 같습니다!
따라서 @RequestHeader을 통해 인증하는 방식으로 컨트롤러단을 수정해주시면 좋을 것 같습니다.

hoonyworld

고생하셨습니다!
현재 강제 예약 삭제를 구현하실 때 POST 메서드로 구현을 하신 것 같습니다.
그런데 해당 api 경로 즉, /force-cancel로 예약을 하지 않았을 때 예약 취소 요청을 보냈을 때, 그리고 예약을 강제 취소하고 다시 예약 취소 요청을 보냈을 때 어떻게 로직이 처리가 되는지 궁금합니다.

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

jwnnoh

고생하셨습니다!
몇 가지 고쳐야 할 부분들 확인해주시면 감사하겠습니다 😄

src/main/java/site/gachontable/gachontablebe/domain/user/domain/repository/UserRepository.java

src/main/java/site/gachontable/gachontablebe/global/error/ErrorCode.java

src/main/java/site/gachontable/gachontablebe/domain/admin/dto/ForceCancelRequest.java

src/main/java/site/gachontable/gachontablebe/domain/admin/usecase/ForceCancel.java

src/main/java/site/gachontable/gachontablebe/domain/waiting/domain/Waiting.java

src/main/java/site/gachontable/gachontablebe/domain/user/domain/User.java

jwnnoh

�우선 고생하셨다는 말씀 드립니다!
또한 기존에 상의되지 않은 기능임을 뒤늦게 파악한 점에 있어서 죄송하다는 말씀 드립니다.

해당 부분같은 경우는 슬랙 및 스크럼을 통해 얘기를 나누도록 하겠습니다! 😄

src/main/java/site/gachontable/gachontablebe/domain/admin/dto/ForceCancelRequest.java

jwnnoh · 2024-05-25T09:21:48Z

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

@@ -35,4 +38,10 @@ public ResponseEntity<JwtResponse> login(@RequestBody AdminLoginRequest request)
        JwtResponse tokens = adminLogin.execute(request.id(), request.password());
        return ResponseEntity.ok(tokens);
    }
+
+    @PostMapping("/force-cancel")
+    public ResponseEntity<String> forceCancel(@RequestBody ForceCancelRequest request) {


Admin의 경우 현재 토큰이 구현되어 있기 때문에 토큰 방식으로 인증 진행하면 될 것 같습니다!
따라서 @RequestHeader을 통해 인증하는 방식으로 컨트롤러단을 수정해주시면 좋을 것 같습니다.

jwnnoh · 2024-05-25T10:11:40Z

src/main/java/site/gachontable/gachontablebe/domain/admin/usecase/ForceCancel.java

기존에 저희 서비스는 최초 호출 이후 5분 경과시 자동 취소 처리되도록 MVP를 설정했었는데,
해당 부분에서 로직의 변경이 불가피할 것 같습니다.

만약 강제 예약 취소를 추가한다고 하면 해당 기능은 Admin이 관리하는 것이기 때문에

Admin이 강제취소하려는 해당 주점의 권한을 가지고 있는지 확인 (FK로 구성)

해당 Admin의 FK에 존재하는 Waiting과 User간의 연관관계 확인
을 수행하는 로직이 추가적으로 이루어져야 할 것 같습니다.

네 알겠습니다. 해당 부분 수정하도록 하겠습니다.

jwnnoh

고생하셨습니다! 몇 가지 의견을 달았으니 확인 부탁드립니다🙏

jwnnoh · 2024-05-27T10:49:44Z

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

+            @ApiResponse(responseCode = "404", content = @Content(schema = @Schema(implementation = ErrorResponse.class))),
+            @ApiResponse(responseCode = "500", content = @Content(schema = @Schema(implementation = ErrorResponse.class)))
+    })
+    @PostMapping("/entered")


rest api 규칙에 따라
/enter 로 uri를 변경하면 좋을 것 같습니다!

jwnnoh · 2024-05-27T10:50:18Z

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

+            @ApiResponse(responseCode = "500", content = @Content(schema = @Schema(implementation = ErrorResponse.class)))
+    })
+    @PostMapping("/entered")
+    public ResponseEntity<String> entered(Authentication authentication, @RequestBody EnteredRequest enteredRequest) {


저희 로직은 Authentication 을 사용하지 않습니다.
따라서 다른 컨트롤러 클래스를 참고하여 @RequestHeader 형식으로 바꿔주시면 좋을 것 같습니다!

jwnnoh · 2024-05-27T10:50:54Z

src/main/java/site/gachontable/gachontablebe/domain/admin/usecase/Entered.java

+
+@Service
+@RequiredArgsConstructor
+public class Entered {


클래스명도 uri에 맞게 수정 부탁드립니다!

jwnnoh · 2024-05-27T10:56:03Z

src/main/java/site/gachontable/gachontablebe/domain/waiting/domain/Waiting.java

@@ -55,6 +55,10 @@ public Waiting(Position waitingType, Integer headCount, Status waitingStatus, Us
        this.pub = pub;
    }

+    public void entered() {


uri에 맞게 메서드명 수정 부탁드립니다!

jwnnoh · 2024-05-27T10:56:58Z

src/main/java/site/gachontable/gachontablebe/global/config/SecurityConfig.java

@@ -50,6 +50,7 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                        authorize
                                .requestMatchers("/v3/api-docs/**", "/swagger-ui/**").permitAll()
                                .requestMatchers("/admin/login").permitAll()
+                                .requestMatchers("/admin/force-cancel").permitAll()


uri명 수정 부탁드립니다!

src/main/java/site/gachontable/gachontablebe/domain/admin/dto/EnteredRequest.java

jwnnoh

클래스명 관련하여 수정 부탁드립니다!

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java

jwnnoh · 2024-05-28T00:35:20Z

src/main/java/site/gachontable/gachontablebe/domain/admin/usecase/EnterUser.java


    private final UserRepository userRepository;
    private final WaitingRepository waitingRepository;
    private final PubRepository pubRepository;
    private final AdminRepository adminRepository;
    private final JwtProvider jwtProvider;

-    public String enter(String authorizationHeader, EnterRequest enterRequest) {
+    public String enterUser(String authorizationHeader, EnterUserRequest enterUserRequest) {


메서드명과 클래스명이 일치해 가독성이 조금 떨어지는 것 같습니다!
적절한 이름으로 변경해주시면 감사하겠습니다..!!

변경 완료하였습니다!

yechan-kim added 4 commits May 23, 2024 21:12

[GTB-31] feat: 강제 삭제 DTO 생성

3a46c9a

[GTB-31] feat: 에러 코드 추가

884c2ef

[GTB-31] feat: 강제 삭제 기능 구현

88ee89e

[GTB-31] fix: 강제 삭제 기능에 대한 권한 부여

47a8628

yechan-kim changed the title ~~Gtb 31~~ Gtb 31 [feat] May 23, 2024

yechan-kim changed the title ~~Gtb 31 [feat]~~ Gtb 31 [feat] 강제 예약 식제 May 23, 2024

yechan-kim changed the title ~~Gtb 31 [feat] 강제 예약 식제~~ Gtb 31 [feat] 강제 예약 식제 구현 May 23, 2024

yechan-kim self-assigned this May 23, 2024

rootTiket reviewed May 24, 2024

View reviewed changes

yechan-kim changed the title ~~Gtb 31 [feat] 강제 예약 식제 구현~~ Gtb 31 [feat] 강제 예약 삭제 구현 May 24, 2024

jwnnoh changed the title ~~Gtb 31 [feat] 강제 예약 삭제 구현~~ GTB-31 [feat] 강제 예약 삭제 구현 May 24, 2024

hoonyworld reviewed May 24, 2024

View reviewed changes

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java Outdated Show resolved Hide resolved

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java Outdated Show resolved Hide resolved

jwnnoh requested changes May 24, 2024

View reviewed changes

yechan-kim added 3 commits May 25, 2024 01:33

[GTB-31] feat: 사용자 예외처리 클래스 생성

f01b682

[GTB-31] feat: 예약 예외처리 클래스 생성

aa95b95

[GTB-31] refactor: 예외 처리 로직 변경

968453f

yechan-kim force-pushed the GTB-31 branch from ac1a4ca to 968453f Compare May 24, 2024 17:10

yechan-kim added 8 commits May 25, 2024 15:20

[GTB-31] feat: 성공 메시지 enum 생성

efdd2a8

[GTB-31] refactor: 성공 메시지 출력 로직 변경

e5b088e

[GTB-31] refactor: 취소 로직 변경

ce23e59

[GTB-31] refactor: API 상세 설명 추가

d55fc52

[GTB-31] refactor: 불필요한 어노테이션 제거

cfbab5b

[GTB-31] refactor: 오류 코드 재정렬 및 정리

6fc774c

[GTB-31] feat: 비어있는 얘약의 수 예외처리 클래스 생성

d7be871

[GTB-31] fix: 유효하지 않은 예약에 대한 오류 수정

5c5b1b1

jwnnoh reviewed May 25, 2024

View reviewed changes

yechan-kim changed the title ~~GTB-31 [feat] 강제 예약 삭제 구현~~ GTB-31 [feat] 입장 완료 구현 May 25, 2024

yechan-kim added 2 commits May 25, 2024 22:55

[GTB-31] refactor: 취소 로직을 완료 로직으로 변경

e1495d6

[GTB-31] rename: ForceCancelRequest.java -> EnteredRequest.java

41ad967

yechan-kim added 5 commits May 27, 2024 18:57

[GTB-31] refactor: API 입력값 수정

d042573

[GTB-31] refactor: 예약 탐색 로직 수정

59448bd

[GTB-31] feat: 존재하지 않는 주점 에러코드 추가

84f0ad1

[GTB-31] feat: 존재하지 않는 주점 예외 클래스 추가

96cef9e

[GTB-31] fix: 중복 예약에 대한 예외 처리

8fd7a9f

jwnnoh requested changes May 27, 2024

View reviewed changes

yechan-kim added 2 commits May 27, 2024 20:29

[GTB-31] refactor: API 인증 방식 변경

1126bd4

[GTB-31] rename: API URL 변경에 따른 로직 이름 변경

73ec4a6

jwnnoh requested changes May 27, 2024

View reviewed changes

src/main/java/site/gachontable/gachontablebe/domain/admin/presentation/AdminController.java Show resolved Hide resolved

[GTB-31] rename: Enter 클래스명과 메서드명을 EnterUser로 변경

e0c23e7

jwnnoh requested changes May 28, 2024

View reviewed changes

[GTB-31] rename: 매서드명 변경 (enterUser -> execute)

3505f8e

yechan-kim merged commit 21a0b8e into develop May 28, 2024

yechan-kim deleted the GTB-31 branch May 28, 2024 00:47

jwnnoh added the ✨ feature 새로운 기능 label Jul 12, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

GTB-31 [feat] 입장 완료 구현 #27

GTB-31 [feat] 입장 완료 구현 #27

yechan-kim commented May 23, 2024 •

edited

Loading

rootTiket left a comment

rootTiket May 24, 2024

rootTiket left a comment

rootTiket May 24, 2024

jwnnoh May 25, 2024

hoonyworld left a comment

jwnnoh left a comment

jwnnoh left a comment

jwnnoh May 25, 2024

jwnnoh May 25, 2024

yechan-kim May 25, 2024

jwnnoh left a comment

jwnnoh May 27, 2024

jwnnoh May 27, 2024

jwnnoh May 27, 2024

jwnnoh May 27, 2024

jwnnoh May 27, 2024

jwnnoh left a comment

jwnnoh May 28, 2024

yechan-kim May 28, 2024

GTB-31 [feat] 입장 완료 구현 #27

GTB-31 [feat] 입장 완료 구현 #27

Conversation

yechan-kim commented May 23, 2024 • edited Loading

1. 무슨 이유로 코드를 변경했나요?

2. 어떤 위험이나 장애를 발견했나요?

3. 관련 스크린샷을 첨부해주세요.

4. 완료 사항

5. 추가 사항

rootTiket left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rootTiket left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hoonyworld left a comment

Choose a reason for hiding this comment

jwnnoh left a comment

Choose a reason for hiding this comment

jwnnoh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jwnnoh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jwnnoh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yechan-kim commented May 23, 2024 •

edited

Loading