refactor: セッション情報を Contenful から取得して表示させるようにする

inouetakuya commented 4 years ago

resolve #183

TODO

Markdown パーサーのライブラリ選定が妥当か
- https://github.com/vuejs-jp/vuefes-2019/pull/204#issuecomment-570887361
コードの記述で気になる点がないか
不要になったコードが削除されているか
本番環境のセッションページと、プレビュー環境のものとで、ビジュアルに差分がないか？
- 本番環境のセッションページには .session-description ul, li にスタイルがないが、ここのスタイルは、既存の「グッズ販売における利益は以下に分割して全額寄付される予定です。」のスタイルと合わせるようにした

セッションの説明、スピーカーの説明は Markdown だが、nuxt generate で生成される HTML では既に Markdown から HTML に変換されている。そのため、ブラウザ表示時の Markdown のパースによるパフォーマンスの劣化は気にしなくてよい 👌

inouetakuya commented 4 years ago

要件としては

セキュアであること
- セッションとスピーカー情報の内容は、運営側で管理できるので XSS のリスクはないが、他の情報をパースするようになったときに、いちいち別のライブラリを使うとかはやりたくないため
ユニバーサルで動くこと
メンテナンスされていること
広く使われていること

あたりを満たすものを探してみて marked が良さそうという結論に至った。

markedjs/marked: A markdown parser and compiler. Built for speed. https://github.com/markedjs/marked

圧倒的な GitHub スター数
速い
ユニバーサルで動く
セキュリティ
- v0.3.9 で XSS 対策がほぼ終わっている
- https://github.com/markedjs/marked/releases/tag/0.3.9 Merry XSSmas :santa:
- その後も引き続きセキュリティ対策を行っている :+1:
CommonMark も GtHub Flavored Markdown もサポートしている
- https://marked.js.org/#/README.md

markdown-it/markdown-it: Markdown parser, done right. 100% CommonMark support, extensions, syntax plugins & high speed https://github.com/markdown-it/markdown-it

showdownjs/showdown: A bidirectional Markdown to HTML to Markdown converter written in Javascript https://github.com/showdownjs/showdown

evilstreak/markdown-js: A Markdown parser for javascript https://github.com/evilstreak/markdown-js

inouetakuya commented 4 years ago

レビューありがとうございます！

コードは問題なさそうですが、 /assets/images/speakers が削除できると思います。

モバイル版の speaker-description のフォントが少し大きめでした（ちなみにデスクトップ版では差分はなかったです）

確認をお願いします！:pray:

inouetakuya commented 4 years ago

レビューありがとうございます！マージします！