-
Notifications
You must be signed in to change notification settings - Fork 52
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.
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
一部のレコードで「市区町村名ローマ字」フィールドが途切れている問題を修正 #123
Conversation
9fbef56
to
844f06a
Compare
9d0a601
to
f55b6dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます。
API に大量に差分が出ているのは JSO もソートされたからという理解でいいでしょうか?
そういうことでしたら OK だと思います!
@champierre コンフリクトの解消だけお願いします 🙇 |
あ、 |
はい、そのとおりです。 |
issue-118.json は必要で、issue-118.csv が不要なので削除します。 |
latest.csv から抜き出してきた csv を patch 用の json 形式に変換するために使用した csv-to-json.js は gist に移したので、削除します。 https://gist.github.com/champierre/ddf80c3079eeade45a9b0f00f2ede75b |
f55b6dd
to
8ca90c6
Compare
ありがとうございます、マージします! |
Closes #118 |
APIの更新もコミットしてしまったため、内容が巨大になってしまっていますが、メインの変更は、patches 以下の変更を入れ込む処理を bin/build-api.js から bin/build.js に移動し、api だけでなく latest.csv および latest.db にも反映させたことです。