-
Notifications
You must be signed in to change notification settings - Fork 177
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
[또링] 웹 기반 체스 게임 3단계 구현 #129
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ttkmw
approved these changes
Apr 29, 2020
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.
안녕하세요~ 임루트입니다.
잘 구현하셔서 코멘트 달 곳을 찾지 못했습니다~!
어프루브하겠습니다.
고생하셨습니다~!^^
woowahan-pjs
approved these changes
Apr 29, 2020
gracefulBrown
pushed a commit
that referenced
this pull request
May 5, 2020
* [또링] 웹 기반 체스 게임 1단계 구현 (#52) * feat : 코드 마이그레이션 * docs : README 작성 * refactor : 테이블 구조 변경 - 단일 게임만 가능하던 구조에서 다중 게임이 가능하도록 수정 * refactor : 예외를 try-catch에서 exception handler로 관리 * refactor : unicode로 변환하는 기능을 service에서 분리 * fix : 적을 잡으면 DB에서 삭제하도록 수정 * refactor : 페이지 렌더링 방식 변경 * refactor : 게임 종료 시 alert를 통해 알림 * fix : í�폰이 대각선으로 움직이는 기능 수정 * refactor : autoBoxing 비용 절감 * refactor : SQLException을 SQLAccessException으로 처리 * refactor : SQLException 발생 시 SQLAccessException에 해당 에러 정보 전달 * refactor : DBConnector에서 에러 발생시 프로그램 종료 * refactor : PieceDao의 save() 반환타입 변경 및 사용하지 않는 메서드 제거 * refactor : 추상적인 메서드 이름을 구체적으로 변경 * docs : 2단계 README 작성 * refactor : Spring Application으로 변경 * refactor : exception handler를 위한 advice 생성 * test : controller 테스트 추가 * test : controller의 save, load 테스트 추가 * [또링] 웹 기반 체스 게임 2단계 구현 (#106) * docs : 2단계 README 작성 * refactor : Spring Application으로 변경 * refactor : exception handler를 위한 advice 생성 * test : controller 테스트 추가 * test : controller의 save, load 테스트 추가 * docs : 3단계 README 작성 * feat : entity 생성 * feat : repository 생성 * feat : 3단계 Spring Data JDBC 적용 * refactor : 서비스 코드 개선 * [또링] 웹 기반 체스 게임 3단계 구현 (#129) * docs : 2단계 README 작성 * refactor : Spring Application으로 변경 * refactor : exception handler를 위한 advice 생성 * test : controller 테스트 추가 * test : controller의 save, load 테스트 추가 * docs : 3단계 README 작성 * feat : entity 생성 * feat : repository 생성 * feat : 3단계 Spring Data JDBC 적용 * refactor : 서비스 코드 개선 * docs : 4단계 README 작성 * feat : 생성된 방 목록을 보여주는 기능 구현 * fix : 게임 id와 움직일 정보를 hidden으로 저장 * refactor : 게임 목록을 보여줄 때 DB에서 게임 ID만 가져오도록 수정 * fix : 점수가 제대로 계산되도록 수정 * refactor : console.log 제거 * refactor : 메서드 이름 변경 * merge from jnsorn * 피드백 반영 * 피드백 반영 Co-authored-by: 또링 <jnsorn@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
잘부탁드립니다~😁