feat: update naver map open api url - Githubissues

zeakd / react-naver-maps

React Navermaps API integration for modern development.

https://zeakd.github.io/react-naver-maps/

118 stars 23 forks source link

feat: update naver map open api url #19

Closed Brew-Brew closed 5 years ago

Brew-Brew commented 5 years ago

Why? 기존 네이버 개발자 센터 클라이언트 아이디는 2019년 4월 19일 까지 사용가능하여 종료 되었고 https://openapi.map.naver.com/openapi/v3/maps.js?clientId 형태로 사용되고 있는 부분이 필요 없어졌으므로 https://openapi.map.naver.com/openapi/v3/maps.js?ncpClientId 와 같은 꼴로 수정되어야 합니다.

스크린샷 2019-06-22 오전 3 21 21

Changes

src/loadNavermapsScript.js 경로에 있던 clientId, ncpClientId로 url 분기를 치던 부분을 ncpClientId로 url이 일괄 적용 되도록 기존 clientId ncpClientId 를 사용하는 사람들이 모두 사용할수 있도록 수정하였습니다.
예제 map api주소를 변경했습니다.

Post actions

예제에 있는 client id가 ncp에서 발급받은 client id 로 교체되어야 합니다.
setUp.js에 있는 아래 부분
```
const MAP_CLIENT_ID = process.env.MAP_CLIENT_ID || 'lvSuBZ0VzLiv7k7YbJXt'
```
본인의 ncp client id로 넣어 테스트한 결과 현재 예제 페이지에 나오고 있지않는 map들 정상 작동

zeakd commented 5 years ago

clientId drop 좋네요. 머지하겠습니다. Thanks! 😸