Conversation
|
すぐ出せる情報のみでrssフィードページを作成しました。
|
|
公式のベストプラクティスによると、podcast側の必須タグは下記3点のようです。
|
追加のタグは必須ではないのですが、Apple Podcastsでおすすめとして紹介されやすくなるようなので追加しました。 cf. https://help.apple.com/itc/podcasts_connect/#/itcb54353390
|
暫定で下記のカテゴリ(サブカテゴリ)を設定してみました。 カテゴリ: Technology |
|
追加したいタグ
|
podcastsに推奨されていたので、ファイルの情報を追加しました。
|
やることに以下の二つが入っているのですが、オンラインで公開しないとテスト・検証・送信はできないのでこのPRのやることからは削除しようと思います。
|
よさそう d(￣ ￣)✨ |
@odlovesq 割と簡単に作れそうだったのでひとまずこんな感じで作ってみました! 🎨 ✅ 他、必要な情報などあればお気軽にご連絡くださ〜い 😆✨
|
nalabjp
left a comment
nalabjp
left a comment
There was a problem hiding this comment.
細かな点をいくつかコメントしましたが、大枠は良さそうに思います！ 👍
PRありがとうございます〜 😌
app/models/podcast.rb
Outdated
| end | ||
|
|
||
| def published_at | ||
| @pubDate ||= exists? ? Time.parse(self.content.lines.second.gsub(/<.+?>/, '').delete('収録日: ')) : '' |
There was a problem hiding this comment.
キャメルケースよりスネークケースの方がrubyっぽいと思いました。
s/@pubDate/@published_at/
There was a problem hiding this comment.
「2行目に収録日が記載されている」というのは仕様になっている感じでしょうか？
仕様がわからなかったのでご存知なら教えてください。
仕様じゃないケースがあるかもという想定を考慮に入れると収録日: で始まる行を探して、それ以降の文字列を切り抜く、という感じかなと思いました。
There was a problem hiding this comment.
「2行目に収録日が記載されている」というのは仕様になっている感じでしょうか？
こちらは Issue に切り出しました ✂️
#407
app/views/podcasts/feed.rss.builder
Outdated
| xml.description full_description "" | ||
| xml.link root_url | ||
| xml.author @author | ||
| xml.copyright "Copyright © 2012-2018 一般社団法人 CoderDojo Japan" |
There was a problem hiding this comment.
@author使えますでしょうか？
"Copyright © 2012-2018 #{@author}"
There was a problem hiding this comment.
s/2018/2019/なので、
"Copyright © 2012-#{Time.current.year} #{@author}"とするのはどうでしょうか？
app/views/podcasts/feed.rss.builder
Outdated
| xml.author @author | ||
| xml.description episode.description | ||
| xml.link @domainname + episode.url | ||
| xml.guid({:isPermaLink => "false"}, @domainname + episode.url) |
There was a problem hiding this comment.
String Interpolationを使いたいと思いました。
"#{@domainname}#{episode.url}"
app/views/podcasts/feed.rss.builder
Outdated
| xml.guid({:isPermaLink => "false"}, @domainname + episode.url) | ||
| xml.itunes :explicit, "clean" | ||
| xml.pubDate episode.published_at.rfc2822 | ||
| xml.enclosure({:url => @domainname + episode.url + ".mp3", :length => episode.filesize, :type => "audio/mpeg"}) |
There was a problem hiding this comment.
String Interpolationを使いたいと思いました。
"#{@domainname}#{episode.url}.mp3"
|
こちら今日中にマージしちゃいたいので、僕の方で指摘点を対応して、さらに修正対応が必要であれば別途リファクタリングする方向で調整してみますね 🔧💨 ✨ |
| xml.language "ja" | ||
| xml.itunes :category, :text => "Technology" do | ||
| xml.itunes :category, :text => "Software How-To" | ||
| xml.itunes :category, :text => "Podcasting" |
There was a problem hiding this comment.
サブカテゴリの追加の仕方ってこれであってるのかな 🤔💭 ちょっと確認してみよ 👀 💨
カテゴリ: Technology
サブカテゴリ: Software How-To, Podcasting
There was a problem hiding this comment.
まぁひとまず検証してどんなデータが出てくるか試してみるのが早そうかな ✅
|
一旦マージして、テスト＆検証していきますね 🔧 💨 ✅ |
|
こちら先ほど Submit しました! 📮 🏃💨 対応ありがとうございます...!! (＞人＜ )✨
|
|
確認遅くなりました！ |
3 participants
背景
TL;DR: DojoCastがPodcastから聴けると嬉しいというご意見がありました！
Fix #363
期待される結果
やること
Podcastでテスト・検証Podcastに送信やらなかったこと