Skip to content

Conversation

7oei
Copy link
Contributor

@7oei 7oei commented Jul 20, 2021

筋肉大好き!!!!!!!

@7oei 7oei requested review from HansRobo and ssr-yuki July 20, 2021 13:57
Copy link
Member

@ssr-yuki ssr-yuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

拝見しましたわ!大変良い見た目になっていましたの.

一方で,以下の懸念事項がございましてよ.

  • 元のアーカイブページを消す必要がありまして?
    • このままではコンパスのページや開催日時等が完全に消えてしまいます.
  • 毎回これを手作業で更新するのはあまりにも工数が多くないかしら?

現状でのおapproveは難しく思いますの.ごめんあそばせ.
@7oei さんがどのような目的でどのようなページを作りたいのかをより詳しく聞かせていただきたいですわね.

それから,コミットメッセージは英語の方がよろしくてよ.ファイル名も改善の余地があるわね.

@7oei
Copy link
Contributor Author

7oei commented Jul 20, 2021

このままではコンパスのページや開催日時等が完全に消えてしまいます.
→イベントのメタページがあります。
発表のメタページにイベントのメタページへのリンクがあります。

@7oei
Copy link
Contributor Author

7oei commented Jul 20, 2021

毎回これを手作業で更新するのはあまりにも工数が多くないかしら?
→それはそう(でも無理そうなレベルでもないので、私が全てやっても良い。)
→ある程度は自動生成も出来そう

@7oei
Copy link
Contributor Author

7oei commented Jul 20, 2021

ふどのような目的でどのようなページを作りたいのかをより詳しく聞かせていただきたいですわね.
→現状のアーカイブページは入り口が小さく、見つけにくい様に感じた→大きなボタンでindex最上部に移動
→アーカイブを見る人はそれが2020の発表か2021の発表かは割とどうでもよくて、どの勉強会で発表されたかもどうでもよくて、どんな発表があったか、どんな資料があるのか一望できた方が良いと感じた→アーカイブページを統合、アーカイブページからのイベント情報の除去(その代わりイベント情報はそれ用のメタページ作って、発表のメタページにリンク作りました。)
→個人的にもっとアーカイブの外からのアクセス性良くなって欲しいなと感じていた(資料のリンクがツイートされてもそこからはアーカイブにアクセス出来ない)→資料毎のアーカイブページを追加、そこにtweetボタンを実装することでメタページがツイートされればそこからその発表があったイベントの情報や他の発表にもアクセスが可能になった。

@7oei
Copy link
Contributor Author

7oei commented Jul 20, 2021

コミットメッセージは英語の方がよろしくてよ.
→以後気を付けます。

@7oei
Copy link
Contributor Author

7oei commented Jul 20, 2021

ファイル名も改善の余地があるわね.
→具体的にどれが良くないか知りたい。数字だけのやつかな?どう直すのがいい?

@7oei 7oei changed the title アーカイブの見やすさの改善、メタページの実装、tweetボタンの実装 Improved archive visibility, metapage implementation, tweet button implementation Jul 22, 2021
@ssr-yuki
Copy link
Member

git commit --amend --author="hoge <[email protected]>"

でコミッターの情報も変えられましてよ!!!

@7oei
Copy link
Contributor Author

7oei commented Jul 31, 2021

修正しやした!

@7oei
Copy link
Contributor Author

7oei commented Jul 31, 2021

@ssr-yuki

@ssr-yuki
Copy link
Member

おえいくんのアカウントと紐付かないのなんで……

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants