Skip to content
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주차 과제 스텝 2 #73

Open
wants to merge 31 commits into
base: joominchul
Choose a base branch
from

Conversation

joominchul
Copy link

No description provided.

joominchul added 20 commits July 8, 2024 15:28
필요 없는 데이터는 주석화 함.
로컬 API로 검색을 하기에 더이상 필요치 않음.
기존의 PlaceAdapter를 카카오 로컬 데이터를 사용하는 DocumentAdapter로 변경
기존 Place 관련 기능들을 Document로 변경
기존 Place 관련 기능들을 Document로 변경
애플리케이션을 통해 키 해시 값 구하고 앱 키 추가
앱을 처음 실행하면 지도 화면을 표시한다.
검색창을 선택하면 검색 화면으로 이동한다. 검색 화면에서 뒤로 가기를 하면 지도 화면으로 돌아온다.
interface RetrofitService {
@GET("keyword.json")
fun requestPlaces(
@Header("Authorization") Authorization: String,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
@Header("Authorization") Authorization: String,
@Header("Authorization") authorization: String,

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 완료했습니다.

@mkSpace
Copy link

mkSpace commented Jul 14, 2024

패키지명이 대문자로 시작하는 부분도 Kotlin Convention에 벗어나는 네이밍 입니다. 그 부분도 같이 수정 부탁드려요

@joominchul
Copy link
Author

패키지명이 대문자로 시작하는 부분도 Kotlin Convention에 벗어나는 네이밍 입니다. 그 부분도 같이 수정 부탁드려요
패키지명은 DTO 같은 폴더명을 말씀하시는 건가요, 아니면 위의 Authorization을 말씀하시는 걸까요?

@mkSpace
Copy link

mkSpace commented Jul 16, 2024

패키지명이 대문자로 시작하는 부분도 Kotlin Convention에 벗어나는 네이밍 입니다. 그 부분도 같이 수정 부탁드려요
패키지명은 DTO 같은 폴더명을 말씀하시는 건가요, 아니면 위의 Authorization을 말씀하시는 걸까요?

Adapter, DBHelper, DTO 등의 폴더명 입니다

@joominchul
Copy link
Author

패키지명이 대문자로 시작하는 부분도 Kotlin Convention에 벗어나는 네이밍 입니다. 그 부분도 같이 수정 부탁드려요
패키지명은 DTO 같은 폴더명을 말씀하시는 건가요, 아니면 위의 Authorization을 말씀하시는 걸까요?

Adapter, DBHelper, DTO 등의 폴더명 입니다

아하! 그렇군요. 이번 주차 과제에서 바꾸도록 하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants