Update PBL002UpsertPlaceData.service.ts

Takapy1 commented 1 year ago

@Ayato-kosaka ごめん、レビューしたの俺なのに追加で修正すべきものがあったから、proposeするわ

Detailsのtypesのadministrative関連はスネークケースで来る
sublocalityはlevelはなく一つしかない

Ayato-kosaka commented 1 year ago

あ、ほんとだ、ごめん！！

Ayato-kosaka commented 1 year ago

sublocality_level は出てるくない？ @Takapy1

curl -XPOST -H "Content-Type:application/json" localhost:3000/PBL002 -d '{"place_id":"ChIJK-moTZORQTURi-csuIHe-MA", "language":"ja"}'[
  { long_name: '３', short_name: '３', types: [ 'premise' ] },
  {
    long_name: '３',
    short_name: '３',
    types: [ 'sublocality_level_4', 'sublocality', 'political' ]
  },
  {
    long_name: '下川端町',
    short_name: '下川端町',
    types: [ 'sublocality_level_2', 'sublocality', 'political' ]
  },
  {
    long_name: '博多区',
    short_name: '博多区',
    types: [ 'sublocality_level_1', 'sublocality', 'political' ]
  },
  {
    long_name: '福岡市',
    short_name: '福岡市',
    types: [ 'locality', 'political' ]
  },
  {
    long_name: '福岡県',
    short_name: '福岡県',
    types: [ 'administrative_area_level_1', 'political' ]
  },
  {
    long_name: '日本',
    short_name: 'JP',
    types: [ 'country', 'political' ]
  },
  {
    long_name: '812-0027',
    short_name: '812-0027',
    types: [ 'postal_code' ]
  }
]

Ayato-kosaka commented 1 year ago

@Takapy1 sublocality も sublocality_level_X もあるみたいだね

[
  { long_name: '１−３', short_name: '１−３', types: [ 'premise' ] },
  {
    long_name: '２',
    short_name: '２',
    types: [ 'sublocality_level_4', 'sublocality', 'political' ]
  },
  {
    long_name: '２丁目',
    short_name: '２丁目',
    types: [ 'sublocality_level_3', 'sublocality', 'political' ]
  },
  {
    long_name: 'みなとみらい',
    short_name: 'みなとみらい',
    types: [ 'sublocality_level_2', 'sublocality', 'political' ]
  },
  {
    long_name: '西区',
    short_name: '西区',
    types: [ 'sublocality_level_1', 'sublocality', 'political' ]
  },
  {
    long_name: '横浜市',
    short_name: '横浜市',
    types: [ 'locality', 'political' ]
  },
  {
    long_name: '神奈川県',
    short_name: '神奈川県',
    types: [ 'administrative_area_level_1', 'political' ]
  },
  {
    long_name: '日本',
    short_name: 'JP',
    types: [ 'country', 'political' ]
  },
  {
    long_name: '220-8173',
    short_name: '220-8173',
    types: [ 'postal_code' ]
  }
]

Ayato-kosaka commented 1 year ago

sublocality は sublocality_level_X を持つでいいんじゃない？

Takapy1 commented 1 year ago

@Ayato-kosaka sublicalityはあるんか、そしたら、subの方はlevel持つ感じでいいかな！スネークケースで！

Ayato-kosaka commented 1 year ago

@Takapy1 オッケー d887c100bd052748637bc366b41810512deced89 で間違いなければMERGE して！

Takapy1 commented 1 year ago

@Ayato-kosaka ごめん、forkした状態からpull requestしてるから、merge権限ない。内容確認したから、こさの方でマージしちゃって！

Ayato-kosaka / spelieve-backend-place

Update PBL002UpsertPlaceData.service.ts #9