fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

ブログ記事の一覧にて、feature タグがついてるブログ記事はメンター・管理者にだけ目印が表示される。 #8186

Open ayu-0505 opened 1 week ago

ayu-0505 commented 1 week ago

Issue

概要

ブログ記事の一覧にて、feature タグがついてるブログ記事は、メンター・管理者にのみ目印であるfeatureという文言が表示されるように変更しました。

変更確認方法

  1. feature/highlight-blogs-with-a-feature-tagをローカルに取り込む
    1. git fetch origin pull/8186/head:feature/highlight-blogs-with-a-feature-tag (2度目以降は上記ブランチのローカル変更に気をつけながら--forceをつけてください)
    2. git switch feature/highlight-blogs-with-a-feature-tag
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. メンターまたは管理者(例: user:komagata, password: testtest)でログインする。
  4. ブログの新規作成ページにアクセスし、タグ入力欄でfeatureを入力し、エンターを押してタグを登録する。
  5. タイトル、本文、サムネイル画像を適当に入力し、公開するボタンをクリックする。
  6. ブログ記事の一覧にアクセスし、先ほど作成したブログ記事に目印である注目の記事の文字が表示されているか確認する。
  7. 一旦ログアウトし、ブログ記事の一覧に再度アクセス。
  8. 先ほど作成したブログ記事に目印である注目の記事の文字が表示されていないか確認する。
  9. メンターまたは管理者以外で再度ログインする(例: user:kimura, password: testtest
  10. ブログ記事の一覧にアクセスし、先ほど作成したブログ記事に目印である注目の記事の文字が表示されていないか確認する。

Screenshot

変更前

tag-testのブログにはfeatureタグをつけており、メンターまたは管理者としてログインしているが、目印は表示されない

スクリーンショット 2024-11-12 12 59 23

変更後

メンターまたは管理者としてログイン時

スクリーンショット 2024-11-19 11 58 36

非ログイン時

スクリーンショット 2024-11-19 11 59 35

管理者外でログイン中

スクリーンショット 2024-11-19 12 09 47
ayu-0505 commented 1 week ago

@machida さん、お疲れ様です🍵

こちらのPRですが、デザインが必要だと思われますがいかがでしょうか。 現在はfeatureという素の文字列がPタグで表示されている状態です。 また、CSSクラスについては周囲の状況から考え、.thumbnail-card__metaのみ記載しております。

ご確認のほどよろしくお願いいたします🙇🏻‍♀️

machida commented 1 week ago

@ayu-0505 デザイン入れてこのブランチにPR出しますー

machida commented 1 week ago

@ayu-0505 ちょっとした変更だったのでこのブランチにpushしましたー。ご確認お願いします🙏 feature 以外にも卒業生インタビューなど特別なタグを用意する予定で、特別なタグを複数持つ記事も出るかもしれないので、それに対応できるようなHTMLとCSSにしました。

ayu-0505 commented 1 week ago

@machida さん、対応いただきありがとうございました🙏

ayu-0505 commented 5 days ago

@hagiya0121 さん、お疲れ様です🍵

もしよろしければこちらのPRのレビューをお願いできますでしょうか。 お忙しい場合は遠慮なくご連絡いただけたらと思います。 ご確認のほどよろしくお願いします🙇🏻‍♀️

hagiya0121 commented 5 days ago

@ayu-0505 了解です!三日以内には終わると思います🙂

ayu-0505 commented 3 days ago

@hagiya0121 さん、確認いただきありがとうございました🍵 修正いたしましたので、またお時間ある時に確認いただけたらと思います🙏