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.

Sign up for GitHub

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

Jump to bottom

外部公開ブログに atom フィードを追加 #7899

Merged
merged 4 commits into from
Jul 15, 2024
Merged

外部公開ブログに atom フィードを追加 #7899

merged 4 commits into from
Jul 15, 2024

Conversation

nishitatsu-dev
Copy link
Contributor

@nishitatsu-dev nishitatsu-dev commented Jun 27, 2024 •
edited
Loading

Issue

概要

  • 外部公開ブログにatomフィードを追加した

変更確認方法

  1. feature/add-atom-feed をローカルに取り込む
  2. atomフィードへのリンクの確認
    1. 外部向けページのフッターに配置されたリンク（Screenshot参照）からatomフィードにアクセスできる事を確認する
  3. RSS / atomリーダーでの表示確認
    1. RSS / atomリーダーを準備し、atomフィードのアドレス(http://localhost:3000/articles.atom)を登録する
      ※動作確認したところ、ReadKit（AppStoreからダウンロード）が、ローカルホストでのatomフィードのアドレスを認識できた。Feedly（ウェブアプリ）は、アドレスを認識できなかった
    2. (development) ブログ | FJORD BOOT CAMP（フィヨルドブートキャンプ）（http://localhost:3000/articles）にアクセスする
    3. 「リーダー」と「元のサイト側」の表示を比較し、表示内容が同じであることを確認する
      表示と操作について
      • WIPの記事は、「メンター」または「管理者」でログインすると「元のサイト側」では確認できるが、atomフィード（＝リーダー）には表示されない
      • atomフィード自体で表示される記事数は、ブログの1ページ当たりの表示数（24） と同じ。ただし、大抵のリーダーでは、一度取り込まれた記事は蓄積されていくので、初期のみ24記事、それ以降は増加する
      • 記事を書く場合、「メンター」または「管理者」でのログインが必要

Screenshot

変更前　（外部向けページのフッター）

image

変更後　（外部向けページのフッター）

image

@nishitatsu-dev nishitatsu-dev force-pushed the feature/add-atom-feed branch from b470c55 to dacc5cc Compare June 27, 2024 15:08
@nishitatsu-dev nishitatsu-dev self-assigned this Jun 28, 2024
@nishitatsu-dev
Copy link
Contributor Author

@machida
外部公開ブログの「RSSフィードのリンク」のデザインをお願いします🙏

@machida machida self-assigned this Jul 2, 2024
@machida
Copy link
Member

machida commented Jul 3, 2024

@nishitatsu-dev おまたせしました！！フッターにリンクを設置しました。

@nishitatsu-dev
Copy link
Contributor Author

@machida
ありがとうございます！！！

@machida machida removed their assignment Jul 4, 2024
@nishitatsu-dev
Copy link
Contributor Author

nishitatsu-dev commented Jul 5, 2024 •
edited
Loading

補足メモ

@nishitatsu-dev nishitatsu-dev requested a review from Shrimprin July 5, 2024 15:46
@nishitatsu-dev
Copy link
Contributor Author

@Shrimprin
レビューをお願いしたいのですが、よろしいでしょうか？
全く急いでおりません。よろしくお願いいたします🙏

実装に関する補足メモです→ #7899 (comment)
不明点あれば、気軽にご連絡ください〜

@Shrimprin
Copy link
Contributor

@nishitatsu-dev
承知いたしました！
RSSについて調べながらになるため1週間ほどお時間ください 🙏

@nishitatsu-dev
Copy link
Contributor Author

nishitatsu-dev commented Jul 9, 2024 •
edited
Loading

@Shrimprin
すみません、１箇所、修正したい所（下記）が見つかりました。修正出来次第、ご連絡します🙇🏻

  • 現状、auto_discovery_link_tag ヘルパーを app/views/articles/index.html.slim に書いていますが、
    レイアウトとレンダリング - Railsガイドによると、「<head>セクションで使用するのが普通」とのことです。
    「ファイルの最終行」に書いてしまっていたのを、<head>セクションの要素がまとめて書いてある「ファイルの頭の方」に移動させようと思います。

@nishitatsu-dev nishitatsu-dev force-pushed the feature/add-atom-feed branch from 14030e5 to 31e85fc Compare July 9, 2024 04:55
@nishitatsu-dev
Copy link
Contributor Author

nishitatsu-dev commented Jul 9, 2024 •
edited
Loading

@Shrimprin
以下を行いましたので、ご確認お願いします🙇🏻

  • auto_discovery_link_tag ヘルパーの位置修正を行いました
  • 他の方のコミットが紛れ込む現象（たまにあるやつです）が出たので、ブランチをリベースしてforce pushしました。
    取り込んであるブランチを捨てて、新しく取り込み直す必要があると思います。（たぶん）
    お手数すみませんが、よろしくお願いします🙏

@Shrimprin
Copy link
Contributor

@nishitatsu-dev
ご連絡ありがとうございます。
修正内容について承知いたしました！

Shrimprin
Shrimprin approved these changes Jul 13, 2024
View reviewed changes
Copy link
Contributor

@Shrimprin Shrimprin left a comment

Choose a reason for hiding this comment

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

お疲れ様です！
問題ないと思いますのでApproveします。

コードとは関係ありませんが、

  • PRがDraft状態のままです。
  • Descriptionの概要が記載されていません。

@nishitatsu-dev nishitatsu-dev marked this pull request as ready for review July 13, 2024 06:40
@nishitatsu-dev
Copy link
Contributor Author

@Shrimprin
レビューありがとうございました！！
いろいろ抜けており、すみません🙇🏻

@nishitatsu-dev nishitatsu-dev requested a review from komagata July 13, 2024 06:48
@nishitatsu-dev
Copy link
Contributor Author

@komagata
レビューをお願い致します🙏

komagata
komagata approved these changes Jul 15, 2024
View reviewed changes
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 9ca0a8a into main Jul 15, 2024
5 checks passed
@komagata komagata deleted the feature/add-atom-feed branch July 15, 2024 18:11
@github-actions github-actions bot mentioned this pull request Jul 15, 2024
Merged
12 tasks
@nishitatsu-dev
Copy link
Contributor Author

@komagata
ありがとうございました〜🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shrimprin Shrimprin Shrimprin approved these changes

@komagata komagata komagata approved these changes

Assignees

@nishitatsu-dev nishitatsu-dev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nishitatsu-dev @machida @Shrimprin @komagata