소환사 닉네임 검색하면 matchId 20개 불러와서 저장하는 코드 #21
Conversation
This file contains hidden or 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
api 호출시 받는 아이디를 이용해 존재유무 확인과 api 호출시 받는 아이디를 이용해 summoner를 불러오는 레포지토리 생성
api 호출시 전체 데이터를 받아오는 dto 생성 엔티티에는 필요한 데이터만 넣기위해 dto를 생성했습니다.
…해 api 호출 데이터를 받아오고 이미 db에 있다면 db 데이터를 리턴하고 없다면 불러온 데이터를 저장하고 리턴하는 코드로 수정
match_id가 존재해도 계속적으로 저장되기 때문에 id를 match_id로 바꿔야 할지 고민해봐야 할것같다
…가해야 하는지 controller를 새로 만들어야 하는지 고민해봐야 할 것 같다.
…atchId 를 저장하는 서비스를 만들었다. 하지만 matchId가 있어도 같은 matchId가 저장되기 때문에 수정이 필요하다.
Contributor
Author
|
전에 닉네임 검색하면 소환사 정보 불러와서 저장하는 코드를 푸쉬했었는데 깜빡하고 pr을 안한것 같습니다. |
s0nnyday
reviewed
Dec 1, 2023
| spring.profiles.include=oauth | ||
| spring.profiles.include=oauth | ||
|
|
||
| riot.api.key=RGAPI-f252410f-7aca-4414-a115-06a9bcac3a7d |
Member
There was a problem hiding this comment.
이 키는 git 에 안올라오도록 git ignore 써야하지않을까 싶습니다!
s0nnyday
reviewed
Dec 1, 2023
| <dependency> | ||
| <groupId>org.apache.httpcomponents</groupId> | ||
| <artifactId>httpclient</artifactId> | ||
| <version>4.5</version> |
Member
There was a problem hiding this comment.
Spring boot 는 버전 정보 안 써주면 알아서 호환되는걸로 맞춰서 지워주는게 좋다고 아는데 어떻게생각하시나요?
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.
matchId가 같아도 중복돼서 저장되기 때문에 코드를 수정해야 한다.
현재 그런 식으로 변경하면 에러가 나서 못하고 있다.