-
Notifications
You must be signed in to change notification settings - Fork 17
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
ニュースページの追加 #196
ニュースページの追加 #196
Conversation
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.
おー!!!良いですね!!!
gatsbyやGraphQLのあれこれが大変かとは思うのですが、できたら他の記事にも応用できると思うので、ぜひ・・・・!!!
残タスクとして残っていた個別記事ページの作成を行いました。 テストで また。 もしくは、既にあるニュース記事にしたい内容をこちらのPRの中に合わせて作成することも可能です。 |
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.
ほぼLGTMです!!
いくつかコメントさせていただきましたが、概ねとても良さそうに思いました!!
まだ記事はできていないので、表示上からは消しつつmainブランチにマージしたいです!
例えばヘッダーからのニュースページへのリンクをコメントアウトしつつ、Seo
コンポーネントでnoindex={true} // TODO: リリース時に外す
などと書くのはどうでしょう・・・!
あ!もしご興味あればなのですが、ogp画像やmarkdown内で画像が使えるようになっていると非常に嬉しいかもです・・・!🙇
ちょっと調べた感じ、OGP画像はこんな感じ、markdown内画像はこんな感じでできるかも・・・・・・?
もちろん一度マージして後続のPRとかでも・・・!!!
src/pages/index.tsx
Outdated
to={"/news/"} | ||
className="has-text-weight-bold is-underlined" | ||
> | ||
ニュース |
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.
変更履歴の左くらいに移動させていただくかもです!
レビューしていただきありがとうございます!!この後再度修正を行います!
かしこまりました!こちらもこの後修正を行っておきます!
こちらもやりたいのですが、今回のPR内容では盛り込みすぎるのではないかと感じたため、別のPRとして盛り込みたいです! |
ありがとうございます!!
おお、検討してくださって嬉しいです!! |
@Hiroshiba |
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.
LGTM!!
ちょっとこちらでフォーマッターにかけさせていただきます!
よろしくお願いします!🙇 |
なぜかpushできなかったので、別PRでマージします! |
内容
関連 Issue
ref #97
スクリーンショット・動画など
その他
voicevox_blog/src/markdowns
に、news
というディレクトリを作成し、その中でマークダウンファイルを作成するとニュース一覧ページに記事のリンクがタイトルと共に展開されます。サンプルで用いた記事のマークダウンはこちらです。