-
Notifications
You must be signed in to change notification settings - Fork 113
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
강원대 BE_고승현_6주차 과제_Step2 #250
Open
chris0825
wants to merge
26
commits into
kakao-tech-campus-2nd-step2:chris0825
Choose a base branch
from
chris0825:step2
base: chris0825
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
지난 주까지 진행했던 kakao-gift-order의 코드를 옮겨왔습니다.
API명세서를 Interface로 만들어 Controller클래스에 상속시킴으로써 코드 가독성을 향상시킴
Front와 협업을 하기 위해 같은 조 BackEnd 끼리 API 명세를 맞춤에 따라 코드를 이에 맞게 수정 - category 조회 컨트롤러 추가 - URI변경 - ResponseBody변경 - Category 인스턴스 변경
- 협업을 위해 바뀐 명세에 맞춰 Front협업 전 관리자 페이지 URI를 수정하였습니다. - 카테고리 객체의 인스턴스가 변경됨에 따라 이를 관리자 페이지에도 반영하였습니다. - 카테고리 등록/수정 페이지에서 이전 페이지로 돌아가는 링크가 작동하지 않던 버그를 수정하였습니다. - option 수정 시 같은 id에 대한 이름도 같은지 비교하여 409 confilct가 발생하였던 오류를 수정하였습니다.
기존 주문을 저장하는 테이블로 사용되었던 테이블의 삭제 시 order예약어와 중복되어 충돌이 발생 따라서 주문을 저장하는 테이블의 이름을 order에서 orders로 변경하여 예약어와 중복을 피하면서 충돌을 해결
font와 협업 중 로그인을 하지 않아도 상품API, 카테고리API, 옵션API를 사용해야 된다고 전달받아서 이 경로의 요청은 인증 헤더를 검사하지 않도록 수정
컨플릭 해결 후 DM 으로 말씀해주시면 머지 작업 도와드리겠습니다. |
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.
주요 변경 사항 - 배포
협업을 위해 바뀐 API명세를 관리자 페이지에도 적용하였습니다.
협업을 위해 상품과 카테고리 API에서 요구되었던 HTTP 요청 시 인증 헤더의 포함 조건을 제외하도록 변경하였습니다.
협업을 위해 CORS를 해결하였습니다.
KAKAO API에서 redirection되는 URI를 AWS EC2서버에 맞게 변경하였습니다.
트러블 슈팅 과정에서 이후 커밋이 작성되었습니다.