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.

Sign up for GitHub

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

Jump to bottom

[feat] [#74] Map 화면 위치 권한 요청 flow 구현 #105

Merged
merged 15 commits into from
Apr 25, 2024
Merged

[feat] [#74] Map 화면 위치 권한 요청 flow 구현 #105

merged 15 commits into from
Apr 25, 2024

Conversation

easyhooon
Copy link
Collaborator

  • 위치 권한 요청 flow 구현
  • 위치 활성화 버튼 추가(해당 버튼을 누르면 내 위치로 지도가 이동되며, 현재 위치한 지점에 오버레이가 추가됩니다)

그밖에

  • [refactor] festivalRepository 와 LikedFestivalRepository 분리 (LikedBooth, Booth repository 처럼)
  • [feat] Menu 화면 관심 축제 아이템 클릭 이벤트 구현(최근에 확인한 관심 축제를 해당 아이템으로 갱신 후, Map 화면으로 이동)

@easyhooon
[add] permission 추가
8b5ea73
@easyhooon
[feat] 앱 설정 이동을 위한 권한 다이얼로그 추가
aef5a7f
@easyhooon
Merge branch 'develop' into feature/location-permission
85502cc
@easyhooon
Merge branch 'develop' into feature/location-permission
9af1a92 
# Conflicts:
#	core/designsystem/src/main/res/values/strings.xml
@easyhooon
[feat] 메뉴 화면 관심 축제 아이템 클릭 이벤트 구현
86dfe8d 
해당 관심 학교가 보여지는 맵 화면으로 이동
@easyhooon
[refactor] festivalRepository 와 LikedRepository 분리
1c4454e
@easyhooon
[refactor] festivalRepository 와 LikedRepository 분리 뷰모델 반영
ec0c51c
@easyhooon
[feat] Map 화면 위치 권한 flow 구현
48821aa 
하지만 별로 맘에 들지 않음
@easyhooon
[chore] code style check success
f82ca9e
@easyhooon
[feat] 개발 편의를 위한 위치 추적 버튼 추가
f80dd14
@easyhooon
[refactor] permissionGranted 변수를 통한 무분별한 권한 요청 실행 방지
d75ed54
@easyhooon
[refactor] 코드 개선
6d0b958 
rememberFusedLocationSource 도입을 통해 ViewModel 내에 context 제거,
FusedLocation 객체 직접 핸들링 하던 로직 제거

권한 요청 flow 개선 (coarse, fine 명시적으로 둘다 요청하도록)

개발의 편의를 위한 LocationButton 활성화(위치 권한이 제대로 수락되었는지 확인을 위함)

location 라이브러리 버전 downgrade, 사용하지 않는 라이브러리 버전 제거
@easyhooon
[chore] code style check success
793f30b
@easyhooon easyhooon added enhancement New feature or request refactor Code restructure for better readability and efficiency labels Apr 24, 2024
@easyhooon easyhooon self-assigned this Apr 24, 2024
This was linked to issues Apr 24, 2024
Map 화면 사용자 위치 권한 요청 플로우 #74
Closed
Menu 화면 구현 #53
Closed
wjdtkdgns777
wjdtkdgns777 approved these changes Apr 25, 2024
View reviewed changes
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

LGTM!! 수고 많으셨습니다!!

@easyhooon easyhooon merged commit d187d8c into develop Apr 25, 2024
1 check passed
@easyhooon easyhooon deleted the feature/location-permission branch April 25, 2024 06:38
@easyhooon easyhooon changed the title [feat] Map 화면 위치 권한 요청 flow 구현 [feat] [#74] Map 화면 위치 권한 요청 flow 구현 Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjdtkdgns777 wjdtkdgns777 wjdtkdgns777 approved these changes

Assignees

@easyhooon easyhooon

Labels
enhancement New feature or request refactor Code restructure for better readability and efficiency
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Map 화면 사용자 위치 권한 요청 플로우 Menu 화면 구현
2 participants
@easyhooon @wjdtkdgns777