Feature custom style #308

fogbar · 2025-04-25T18:20:17Z

iOS, AOS 모두 대응 및 테스트 완료 하였습니다.

*네이버 공식 참고 문서

가이드 : https://guide.ncloud-docs.com/docs/application-maps-styleeditor-connection-vpc
ㄴ AOS : https://navermaps.github.io/android-map-sdk/guide-ko/2-4.html
ㄴ iOS : https://navermaps.github.io/ios-map-sdk/guide-ko/2-4.html

관련 이슈
ㄴ #307

note11g · 2025-04-27T06:03:00Z

코드는 확인하였습니다. 기존 구현대로 잘 구현해주신 것 같아, 좋아보입니다.

완료 콜백(android 기준 onCustomStyleLoaded, onCustomStyleLoadFailed)의 구현이 필요할지 검토가 필요합니다.
1.4.0에서 새로 추가될 콜백에는 지도 로드가 완료되면 호출되는 onMapLoaded가 존재하는데,
이들의 실행 순서가 어떻게 되는지, 어떻게 쓰일 수 있는지(용례)를 고려하여 onCustomStyleLoaded/onCustomStyleLoadFailed 를 추가할지, 아니면 styleId가 있을 때만 다른 콜백과 엮어 호출하게 할지를 결정해야 할 듯합니다.
다른 작업 이후에 다시 한번 살펴보겠습니다.

fogbar · 2025-04-27T08:04:53Z

넵, 확인하였습니다.

custom style의 loaded, failed 콜백에 관하여는 일부러 구현하지 않았습니다.
해당 콜백 구현시 말씀주신 부분들과 같이 고려해야할 부분이 많아 향후 구현을 하게 되더라도 단계 별로 구현될 수 있도록 처리하고자 하였습니다.

긍정적인 검토 감사합니다.

…#229

…get size) with null value on `size` argument for `NOverlayImage.fromWidget` (resolve note11g#304)

note11g · 2025-05-18T08:06:57Z

*auth api의 오류 콜백을 지도로 옮길 것인지 검토

=> 지도로 옮기는 것이 라이프사이클 상 더 명료해보임. 다른 이슈로 분리. (#316)

…ndler.onMapLoaded` too.

…r`, `NaverMapViewEventDelegate`)

…rolSender`, `NaverMapView.setMapEventListeners`)

…lled

…olHandler.onMapLongTapped` too.

…ender`, `NaverMapViewEventDelegate`)

…ControlSender`, `NaverMapView.setMapEventListeners`)

…Tapped`) with snackbar

…on `SdkInitializer.authorized.onAuthFailed` invocation

…on` on `SdkInitializer.onAuthFailedListener.onAuthFailed` invocation

…on `NaverMapController.onCustomStyleLoadFailed.onCustomStyleLoadFailed` invocation

…on` on `NaverMapController.onCustomStyleLoadFailed.onCustomStyleLoadFailed` invocation

note11g · 2025-06-20T09:51:27Z

@fogbar 예외처리를 구현하였습니다.
추가로 style id 동적 변경시 콜백 재설정 코드에 대해서도 구현할 예정입니다.
해당 부분 직접 구현하고 계시는지 확인이 필요합니다.

fogbar · 2025-06-22T06:11:12Z

@note11g 답변이 늦어져 죄송합니다.

아직 구현 중인 부분은 없습니다. 지금 작업에 들어가고자 합니다.
다만, 이미 구현을 다 마치신 것 같아 제가 추가 구현할 부분이 있을지 잘 모르겠습니다.

지금 pull 받고 소스 검토 해보겠습니다.

fogbar · 2025-06-22T10:10:38Z

@fogbar 예외처리를 구현하였습니다. 추가로 style id 동적 변경시 콜백 재설정 코드에 대해서도 구현할 예정입니다. 해당 부분 직접 구현하고 계시는지 확인이 필요합니다.

style id 동적 변경시 콜백 재설정 관련하여 작업 중에 있습니다. 다만, null 로 할당하게 될 경우 Android 기준 customStyleId 관련 메소드가 호출되지 않는 것을 확인하였습니다.

현재까지 확인한 바 NaverMapOptionApplier 의 rawCustomStyleId 를 Any? 로 변경하고 null 처리가 가능하도록 구현이 필요하지만, 이를 위해서는 ApplyUtils 등 관련 로직들에 대한 파악 후 수정이 필요할 것 같습니다. 따라서 시간이 좀 걸리는 작업으로 판단됩니다.

혹시, 이와 관련하여 미리 파악해 두시거나 구현 방법에 대한 방향성이 잡히신게 있으실까요? 제가 방향성을 잘못 잡아 시간이 딜레이 될까 질문드립니다.

note11g · 2025-07-07T08:05:09Z

@fogbar 추가로 진행하신 작업이 있으시다면 코멘트 부탁드립니다.
없으시다면, 기능 릴리즈를 위해 마무리 작업은 제가 진행하도록 하겠습니다.

…NaverMapViewOptions`.

…re-custom-style # Conflicts: # android/src/main/kotlin/dev/note11/flutter_naver_map/flutter_naver_map/applier/option/NaverMapApplierImpl.kt

note11g · 2025-07-13T12:06:08Z

ApplyUtil을 비롯하여 NaverMapOption의 프로퍼티들이 null 값을 커버할 수 있도록 수정하였습니다. (6f40c99 ~ f285005)

…erMapController.remove` with override dispose method

…Id dynamically

…ynamically

…pController.removeChannel` with override dispose method

note11g · 2025-07-13T14:21:33Z

style 동적 변경시에도 콜백이 활성화되도록 수정하였습니다. (b2a4d7b, c1ddc1f)

note11g · 2025-07-13T14:28:25Z

실기기/에뮬레이터 모두 테스트 완료되어, 병합합니다.

related: #308

fogbar · 2025-07-22T19:34:06Z

@fogbar 추가로 진행하신 작업이 있으시다면 코멘트 부탁드립니다. 없으시다면, 기능 릴리즈를 위해 마무리 작업은 제가 진행하도록 하겠습니다.

답이 늦어 죄송합니다. 몇 주간 개발을 건드릴 시간이 없어 추가 작업이 없었습니다.
감사합니다.

note11g · 2025-07-23T08:03:58Z

@fogbar 기여에 감사드립니다. 고생 많으셨습니다:)

fogbar added 2 commits April 26, 2025 01:57

iOS - customStyle 대응

6423b12

Android - customStyle 대응

a4f42d9

note11g changed the base branch from main to dev/1.4.0 April 26, 2025 06:15

note11g requested review from Copilot and note11g April 26, 2025 06:15

note11g linked an issue Apr 26, 2025 that may be closed by this pull request

네이버지도 My Style 기능 지원 관련 질문 #307

Closed

This comment was marked as outdated.

Sign in to view

note11g self-assigned this Apr 27, 2025

note11g linked an issue Apr 27, 2025 that may be closed by this pull request

prepare 1.4.0 #301

Closed

11 tasks

note11g removed a link to an issue Apr 27, 2025

prepare 1.4.0 #301

Closed

11 tasks

note11g mentioned this pull request Apr 27, 2025

prepare 1.4.0 #301

Closed

11 tasks

note11g added 5 commits May 10, 2025 16:17

[Chore] new readme test.

177ae6b

[Chore] new readme draft

84c03ea

[Chore] readme update (add migration notice)

56b08c0

[Update, iOS] change InfoWindow ui match with android and fix note11g…

f35578e

…#229

[Update] add support dynamic widget image(automatically detecting wid…

ec2a5be

…get size) with null value on `size` argument for `NOverlayImage.fromWidget` (resolve note11g#304)

note11g and others added 11 commits May 18, 2025 17:23

[Add] add onMapLoaded on NaverMap widget. and `_NaverMapControlHa…

36175c2

…ndler.onMapLoaded` too.

[Impl, iOS] impl onMapLoaded on iOS Platform (`NaverMapControlSende…

57f1b09

…r`, `NaverMapViewEventDelegate`)

[Impl, Android] impl onMapLoaded on Android Platform (`NaverMapCont…

a8512bc

…rolSender`, `NaverMapView.setMapEventListeners`)

[Change] change onMapLoaded calling time when after onMapReady ca…

208a3ee

…lled

[Chore] add example to use onMapLoaded

f77cb6c

Add NaverMap.onMapLoaded (note11g#315)

635622d

[Add] add onMapLongTapped on NaverMap widget. and `_NaverMapContr…

337080b

…olHandler.onMapLongTapped` too.

[Impl, iOS] impl onMapLongTapped on iOS Platform (`NaverMapControlS…

305f834

…ender`, `NaverMapViewEventDelegate`)

[Impl, Android] impl onMapLongTapped on Android Platform (`NaverMap…

b58b203

…ControlSender`, `NaverMapView.setMapEventListeners`)

[Chore] add example onMapLongTapped with snackbar

da34d37

[Chore] add example display on tap listeners(onMapTapped, `onSymbol…

a55b580

…Tapped`) with snackbar

note11g added 6 commits June 20, 2025 18:44

[Add, iOS] add common native exception type NFlutterException

51db556

[Add, Android] add common native exception type NFlutterException

9b89821

[Update, iOS] using common native exception type NFlutterException …

d8b88b3

…on `SdkInitializer.authorized.onAuthFailed` invocation

[Update, Android] using common native exception type `NFlutterExcepti…

8a0a0df

…on` on `SdkInitializer.onAuthFailedListener.onAuthFailed` invocation

[Update, iOS] using common native exception type NFlutterException …

f0c1dec

…on `NaverMapController.onCustomStyleLoadFailed.onCustomStyleLoadFailed` invocation

[Update, Android] using common native exception type `NFlutterExcepti…

0c56a8a

…on` on `NaverMapController.onCustomStyleLoadFailed.onCustomStyleLoadFailed` invocation

kotlin/DefaultTypeConverter asString 관련 삭제

69d2ed4

This comment was marked as outdated.

Sign in to view

note11g added 5 commits July 13, 2025 20:31

[Update, Android] Support nullable options in map configuration

6f40c99

[Update] Support sending nullable value on NPayload.make constructor.

4fea3d8

[Update] Support sending to native(with NPayload) nullable value on `…

ad8fcf0

…NaverMapViewOptions`.

[Update, iOS] Support nullable options in map configuration

f285005

Merge remote-tracking branch 'fogbar/feature-custom-style' into featu…

12465ac

…re-custom-style # Conflicts: # android/src/main/kotlin/dev/note11/flutter_naver_map/flutter_naver_map/applier/option/NaverMapApplierImpl.kt

note11g added 4 commits July 13, 2025 22:32

[Chore, Android] add NaverMapControlSender.dispose and migrate `Nav…

7ca084e

…erMapController.remove` with override dispose method

[Fix, Android] Preserve customStyleCallback when updating customStyle…

b2a4d7b

…Id dynamically

[Fix, iOS] Preserve customStyleCallback when updating customStyleId d…

c1ddc1f

…ynamically

[Chore, iOS] add NaverMapControlSender.dispose and migrate `NaverMa…

5d18d31

…pController.removeChannel` with override dispose method

note11g merged commit 577ed57 into note11g:feature/custom_style Jul 13, 2025

note11g added a commit that referenced this pull request Jul 14, 2025

[Merge PR] Merge to dev/1.4.0 branch (#331)

4e18d82

related: #308

note11g mentioned this pull request Jul 14, 2025

Release 1.4.0-dev.3 #332

Merged

github-actions bot mentioned this pull request Jul 15, 2025

Release 1.4.0 #337

Merged

Feature custom style #308

Feature custom style #308

Uh oh!

Conversation

fogbar commented Apr 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

note11g commented Apr 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fogbar commented Apr 27, 2025

Uh oh!

note11g commented May 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

note11g commented Jun 20, 2025

Uh oh!

fogbar commented Jun 22, 2025

Uh oh!

fogbar commented Jun 22, 2025

Uh oh!

This comment was marked as outdated.

note11g commented Jul 7, 2025

Uh oh!

note11g commented Jul 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

note11g commented Jul 13, 2025

Uh oh!

note11g commented Jul 13, 2025

Uh oh!

fogbar commented Jul 22, 2025

Uh oh!

note11g commented Jul 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

fogbar commented Apr 25, 2025 •

edited

Loading

note11g commented Apr 27, 2025 •

edited

Loading

note11g commented May 18, 2025 •

edited

Loading

note11g commented Jul 13, 2025 •

edited

Loading