zeakd / react-naver-maps

React Navermaps API integration for modern development.
https://zeakd.github.io/react-naver-maps/
119 stars 23 forks source link

fix: react-naver-map 에 기재된 다른 provider 명 변경 #107

Closed FrontLeejonghun closed 11 months ago

FrontLeejonghun commented 1 year ago
스크린샷 2023-07-13 오전 11 44 00 스크린샷 2023-07-13 오전 11 44 22
zeakd commented 1 year ago

@FrontLeejonghun 님 코드들 살펴보시고 PR 주셔서 감사합니다.

react-naver-maps 설계에서 navermapsNaverMap 을 분리해서 사용하고 있습니다. navermaps: window.naver.maps 와 동일한 네임스페이스 NaverMap: 지도 컴포넌트 (Map 이 js 기본 브라우저객체와 충돌하므로 NaverMap 으로 처리)

navermaps 가 전체 소문자인 이유는 이렇습니다.

Provider는 이런 이유에서 navermaps namespace 를 제공한다는 의미로 NavermapsProvider 로 사용되었습니다

또한 react-naver-maps 의 Interface 를 바꾸는 것은 라이브러리 사용자 전체의 Breaking Change 를 야기하기 때문에 신중하게 정하고 크게 변경하지 않는 방향으로 생각하고 있습니다.

그래서 이 부분은 Interface 를 수정하는 것이 아니라 Example의 문서의 수정이 필요할 것 같아요. 🙇 감사합니다.