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

meeup:gen_index と meetup:gen_report の変更提案 #1430

Open
muryoimpl opened this issue Mar 29, 2024 · 3 comments
Open

meeup:gen_index と meetup:gen_report の変更提案 #1430

muryoimpl opened this issue Mar 29, 2024 · 3 comments

Comments

@muryoimpl
Copy link
Contributor

  1. meetup:gen_index 実行時、Gather を使わなくなったので、Gather の選択肢を消す or デフォルトを Zoom にする
  2. meetup:gen_report で出力時、published: false にしているが開催後に出力が多いので、published: true でもいいのでは?

2 については手で書き換えてもらっていると思うので、気にすることが 1 つ減るのはいいことなのではないかと。

@nagutabby
Copy link
Member

index.mdをRakeタスクで生成したときはpublishedがtrueになっていて、report.mdのときはfalseになっているんですよね
どちらかに統一できると勘違いが減ると思います

@nagutabby
Copy link
Member

reportページへのリンクを表示するかどうかを判断する処理でreport.mdのpublishedフロントマターの値を見に行くべきでしたが、その処理を実装する気力がなくPRに含められませんでした
今後、その部分を追加する予定です

@nagutabby
Copy link
Member

#1431 を参照

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

No branches or pull requests

2 participants