Skip to content
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

[🛠️Refactor]:NLP関連のJSONカラムでkeyを日付にしているものをやめる #568

Open
usuyuki opened this issue Aug 10, 2023 · 1 comment
Labels
🛠️refactor リファクタリング周り

Comments

@usuyuki
Copy link
Member

usuyuki commented Aug 10, 2023

🛠️ 概要

現状DBに保存しているJSON形式で

{
    "2023-04": 4957,
    "2023-05": 21948,
    "2023-06": 20532,
    "2023-07": 21940,
    "2023-08": 742
}

こういうkeyを日付にしているケースがかなりある

たしかにパット見ではわかりやすいが

[{
 "date":"2023-04"
"count":4957,
}]

にしたい。

なぜ必要

OpenAPIの定義でkeyに情報を与えられないので、フロントエンドで型のあるくらしを妨げる
myimage

備考

データ構造が変わるので全部かえる必要がある

@usuyuki usuyuki added the 🛠️refactor リファクタリング周り label Aug 10, 2023
@usuyuki usuyuki changed the title [🛠️Refactor]:NLP関連でkeyを日付にしているものをやめる [🛠️Refactor]:NLP関連のJSONカラムでkeyを日付にしているものをやめる Aug 10, 2023
@usuyuki
Copy link
Member Author

usuyuki commented Aug 10, 2023

これChart.jsでグラフレンダリングするときにlabelとdeta取り出す時、key:valueで入ってるとobject.keysとobject.values使うだけなので楽だからこのデータ形式にしてた説がある

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️refactor リファクタリング周り
Development

No branches or pull requests

1 participant