-
Notifications
You must be signed in to change notification settings - Fork 1
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: 교육 결승진출자, 우승자 계산, 재전송 API 구현 #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰남겼습니다 ~
src/main/java/org/cotato/csquiz/api/education/controller/EducationController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/common/websocket/WebSocketHandler.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/domain/education/service/KingMemberService.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/domain/education/service/KingMemberService.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/api/socket/dto/QuizStopResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/domain/education/service/KingMemberService.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Endpoint와 관련된 리뷰 남겼습니다!
소켓 전송과 같은 경우엔 멱등성의 관점에서 GET요청이 서버의 상태를 바꿀 수도 있다는 점에서 부적절하다고 하네요
추가로 전반적으로 endpoint에 동사를 최소화하는게 좋을 것 같은데 현재 SocketController
가 존재하는데 해당 부분 외적인 곳에서 데이터를 전송하는 API가 있는게 괜찮을지 고민이 되네요..
src/main/java/org/cotato/csquiz/api/education/controller/EducationResultController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/api/education/controller/EducationResultController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/api/education/controller/EducationResultController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cotato/csquiz/api/education/controller/EducationResultController.java
Outdated
Show resolved
Hide resolved
* Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github action에 키를 주입하는 방식 변경 * Chore: application.yml 환경 변수 세팅 단계 추가 * Chore: application.yml 환경 변수 세팅 단계 추가 * docs: Update Feature Issue Template * docs: Update Fix Issue Template * docs: PR 템플릿 생성 * docs: PR 템플릿 이모지 제거 * Feature: 교육 결승진출자, 우승자 계산, 재전송 API 구현 (#3) * feat: 교육 결승진출자 계산, 재전송 API 구현 * feat: 교육 결승진출자 계산, 재전송 API 구현 * feat: kingking member, winner 계산 + 전송 API 구현 * feat: 이미 우승자가 계산됐으면 exception 발생 * refactor: 코드 리뷰를 기반으로 코드 수정 * refactor: 컨트롤러, 서비스 메소드 위치 변경 --------- Co-authored-by: GiHun Nam <52378919+gikhoon@users.noreply.github.com>
* Chore: update code-deploy-app-name (#6) * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 (#7) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치명 확인을 위한 스크립트 경로 수정 (#8) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 Git 설정 (#9) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: JAR파일 경로 확인을 위한 와일드카드 설정 (#10) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: JAR파일 생성 전 빌드 작업을 추가 (#11) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 빌드 후 JAR파일의 이름을 변경하는 방식 채택 (#12) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: 테스트 서버 jdbc url 수정 (#13) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Fix: DDL_AUTO 오타 수정 (#14) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github Actions에 키를 주입하는 방식 변경 (#15) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github action에 키를 주입하는 방식 변경 * Chore: 기본 Yaml 파일 환경 변수 세팅 과정 추가 (#16) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github action에 키를 주입하는 방식 변경 * Chore: application.yml 환경 변수 세팅 단계 추가 * Chore: yaml 파일 세팅 과정 오타 수정 (#17) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github action에 키를 주입하는 방식 변경 * Chore: application.yml 환경 변수 세팅 단계 추가 * Chore: application.yml 환경 변수 세팅 단계 추가 * Feature: 교육 결승진출자, 우승자 계산, 재전송 API 구현 (#19) * Chore: update code-deploy-app-name * Chore: set overwrite * Chore: 배포 스크립트 분기 로그 추가 * Chore: 브랜치 확인을 위한 경로 이동 스크립트 추가 * Chore: 작업 디렉토리 신뢰를 위한 git 설정 * Chore: jar파일 경로 구체화 * Chore: jar파일 생성 전 build를 선행 작업으로 추가 * Chore: 정상 build 후 JAR파일 이름을 변경하는 방식으로 수정 * Chore: test server jdbc url 수정 * Chore: test server jdbc url 수정 * Chore: Github action에 키를 주입하는 방식 변경 * Chore: application.yml 환경 변수 세팅 단계 추가 * Chore: application.yml 환경 변수 세팅 단계 추가 * docs: Update Feature Issue Template * docs: Update Fix Issue Template * docs: PR 템플릿 생성 * docs: PR 템플릿 이모지 제거 * Feature: 교육 결승진출자, 우승자 계산, 재전송 API 구현 (#3) * feat: 교육 결승진출자 계산, 재전송 API 구현 * feat: 교육 결승진출자 계산, 재전송 API 구현 * feat: kingking member, winner 계산 + 전송 API 구현 * feat: 이미 우승자가 계산됐으면 exception 발생 * refactor: 코드 리뷰를 기반으로 코드 수정 * refactor: 컨트롤러, 서비스 메소드 위치 변경 --------- Co-authored-by: GiHun Nam <52378919+gikhoon@users.noreply.github.com> --------- Co-authored-by: GiHun Nam <52378919+gikhoon@users.noreply.github.com>
No description provided.