Skip to content

ブログ記事のタグ入力ショートカットを作成 #8557

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lsesmpmido
Copy link
Contributor

@lsesmpmido lsesmpmido commented Apr 14, 2025

Issue

概要

いくつかの重要なタグをタイポしないように選択肢から入力できるようにしました。

変更確認方法

  1. feature/add-shortcut-for-blog-tagをローカルに取り込む
    1. git fetch origin feature/add-shortcut-for-blog-tag
    2. git switch feature/add-shortcut-for-blog-tag
  2. foreman start -f Procfile.devでローカル環境を立ち上げる
  3. 管理者・メンターアカウントでログインする
    • 例: ユーザー名: komagata、パスワード: testtest
  4. 記事の一覧(http://localhost:3000/articles )から記事の新規作成、または編集ページに移動する
  5. タグ入力欄の下にある「タグ入力」の項目からタグを選択し、タグ入力欄にタグが反映されることを確認する
  6. 記事を保存し、「タグ入力」から選択したタグが実際に保存できていることを確認する

Screenshot

変更前

image

変更後

レコーディング 2025-04-15 050050

@lsesmpmido
Copy link
Contributor Author

@machida さん
お疲れ様です!

記事を作成する際に重要なタグをボタンで選択できるようにいたしましたのでボタンのデザイン付けをお願いしたいです。
ご確認よろしくお願いいたします🙇‍♂️

レコーディング 2025-04-14 211221

@machida
Copy link
Member

machida commented Apr 14, 2025

@lsesmpmido デザイン入れましたー

@lsesmpmido lsesmpmido marked this pull request as ready for review April 15, 2025 00:13
@lsesmpmido
Copy link
Contributor Author

@Ryooo-k さん
お疲れ様です!
こちらのレビューをお願いしたいのですがよろしいでしょうか?
全然急ぎではありませんので、ご検討いただけるとうれしいです
よろしくお願いしますm(_ _)m

@lsesmpmido lsesmpmido requested a review from Ryooo-k April 18, 2025 00:01
@Ryooo-k
Copy link
Contributor

Ryooo-k commented Apr 18, 2025

@lsesmpmido
了解しました🫡レビュー完了後、ご連絡しますね👍

Copy link
Contributor

@Ryooo-k Ryooo-k left a comment

Choose a reason for hiding this comment

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

@lsesmpmido
確認しました!テストに関して、一点だけコメントしてますのでご確認お願いします!
久しぶりのReactだったので依存配列やらクリーンアップ関数など忘れていたので勉強になりました👍

fill_in 'タイトル', with: 'ショートカットボタンからのタグ追加テスト'
fill_in '本文', with: 'タグショートカットボタンのテストです'
find('button.js-tag-input-button', text: '注目の記事').click
find('.tagify__input')
Copy link
Contributor

@Ryooo-k Ryooo-k Apr 22, 2025

Choose a reason for hiding this comment

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

質問させてください!ここのfindはタグ追加まで待機することが目的ですか?ちょっと用途が分からなかったので教えてください🙇

Copy link
Contributor Author

Choose a reason for hiding this comment

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

すみません、こちらは'.tagify__input'の箇所に選択したタグが入力されてるかを確認する意図で書いたつもりだったんですが、たしかにこの書き方では不適切でしたねassert_selector('.tagify__tag', text: '注目の記事')で確認できるように修正いたしました

@lsesmpmido lsesmpmido force-pushed the feature/add-shortcut-for-blog-tag branch from 105130d to 997d25a Compare April 22, 2025 02:57
@lsesmpmido
Copy link
Contributor Author

@Ryooo-k さん
レビューありがとうございます!
ご指摘いただいた箇所について修正いたしました!
お手すきの際に改めて確認をお願いいたします!

Copy link
Contributor

@Ryooo-k Ryooo-k left a comment

Choose a reason for hiding this comment

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

@lsesmpmido
確認しましたー!OKです🙆

@lsesmpmido
Copy link
Contributor Author

@Ryooo-k さん
レビューありがとうございました!

@komagata さん
Approveをいただいたので、お手すきの際にレビューの確認をお願いいたします

@lsesmpmido lsesmpmido force-pushed the feature/add-shortcut-for-blog-tag branch from 997d25a to 79f38ac Compare April 28, 2025 02:07
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.

3 participants