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

/api/v1/radios/のレスポンスを変更する #357

Open
kobakazu0429 opened this issue Jun 9, 2019 · 7 comments
Open

/api/v1/radios/のレスポンスを変更する #357

kobakazu0429 opened this issue Jun 9, 2019 · 7 comments

Comments

@kobakazu0429
Copy link
Member

kobakazu0429 commented Jun 9, 2019

#291 RadioCardContentのために

現在

[{
...
"description": "<p>4月に1年生歓迎会が行われました</p> <p>そこで出会った1年生はとてもアクティブな子達が多いこと、新入生歓迎会でやったこと、高専に入って楽しかったことなどについて、<br> 幼稚園から一緒の2人が元気にお届けします!</p> <p>紹介した高専日誌</p> <ul> <li><a href="https://www.kure-nct.ac.jp/newdiary/2019/04/21_2.html">1年生歓迎会の開催</a></li> </ul>"
...
}]

こんな感じ

[{
  "description": {
    "text": "string",
    "紹介した高専日誌": "string | null",
    "紹介したイベント": "string | null",
    "関連リンク": "string |null"
  }
}]
@kobakazu0429 kobakazu0429 changed the title /api/v1/radiosno /api/v1/radios/のレスポンスを変更する Jun 9, 2019
@kobakazu0429
Copy link
Member Author

#291 RadioCardContent はスクロールすることで解決

@kobakazu0429
Copy link
Member Author

kobakazu0429 commented Nov 1, 2019

#288 はどうしようかな
頑張ればいけなくない気もするけど、、、

@euglena1215 上みたいにレスポンスとDB構造の変更についてどう思いますか?
データ移行がめんどくさいし、影響範囲がでかいなって感じです

@euglena1215
Copy link
Member

データ移行はぽちぽち頑張るしかないんじゃないかなという気がする

リニューアルが完了して完全移行するまでは全部入りcolumnと内容ごとのcolumnを両方保存する形にするのがよさそう?

@kobakazu0429
Copy link
Member Author

全部入りcolumnと内容ごとのcolumnを両方保存する形

これが一番安全そうですね

@kobakazu0429
Copy link
Member Author

そもそも #288 に本当にdescriptionがいるのか、というのがあると思って、タイトルだけでもいいのかなって思いました

@euglena1215
Copy link
Member

この変更がリニューアルのボトルネックになるのであれば切ってよいと思ってる!
リニューアルを出すことを第一に考えたい

@kobakazu0429
Copy link
Member Author

#288 についてはとりあえず「タイトルだけ表示する」で対応します!

@kobakazu0429 kobakazu0429 mentioned this issue Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants