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

テストデータにアイコンが長方形のユーザーを追加 #3622

Merged
merged 3 commits into from
Dec 4, 2021

Conversation

taka61
Copy link
Contributor

@taka61 taka61 commented Nov 25, 2021

issue: #3571

概要

テストデータに長方形のユーザーアイコンを登録したユーザーを追加する

使用する画像

tatenoiconのコピー

追加後

tatenoiconという名前のユーザーを追加しました。

スクリーンショット 2021-11-25 13 23 13

アイコンを見る限り縦長表示ではありませんが、「新しいタブで画像を開く」で画像の確認を行うと
町田さんに用意して頂いた縦長アイコンと一致します。
スクリーンショット 2021-11-25 13 23 30

注意点

スクリーンショット 2021-11-25 17 58 23

Binary file not shown.が表示され、追加分の画像が表示されませんが、矢印部分を押すと差分が表示され画像の確認ができます!

@taka61 taka61 self-assigned this Nov 25, 2021
@taka61 taka61 changed the title テストデータに長方形のユーザーアイコンを登録したユーザーを追加 テストデータにアイコンが長方形のユーザーを追加 Nov 25, 2021
@taka61 taka61 marked this pull request as ready for review November 25, 2021 09:04
@taka61
Copy link
Contributor Author

taka61 commented Nov 25, 2021

@yoshitaku-jp
お手数ですが、お時間のある際にこちらのレビューをお願いします!🙏

@yoshitaku-jp
Copy link
Contributor

@taka61 さん
遅くなってすみません!

私の方で上記ユーザを、ユーザ一覧から見つけることができませんでした。。。ログインもできず…
push忘れなどありませんでしょうか?また、再現方法も載せていただくことは可能でしょうか?

私の方でおこなったことは

  1. PRブランチに移動
  2. bin/setupを実行
  3. rails sを実行

image

@taka61
Copy link
Contributor Author

taka61 commented Nov 27, 2021

@yoshitaku-jp さん
確認頂きありがとうございます!

「ユーザ一覧から見つける」に関しては以下の手順で確認をお願いします。

  1. PRブランチに移動後、rails db:seedを実行するとDBの内容が開発環境に反映されます。
  2. rails sで起動し、ユーザー一覧から新しく追加したtatenoiconユーザーが見つけられると思います。
    (説明が分かりづらい場合、日報の方が分かりやすいかもしれないのでこちらをご確認下さい🙏)

「ログイン」に関しては確認したところ私の方でも出来なかった為、解決し次第再度改めて手順を載せさせて頂きます💦
よろしくお願いします🙏

@yoshitaku-jp
Copy link
Contributor

@taka61 さん
すみません!rails db:seedしていませんでした!bin/setupの中でやっていると思っていました:bow:
こちらの手順も簡単ですが明記しておいてよかったです…!

そして、無事にユーザを見つけることができました!大丈夫だと思います!

image

image

@taka61
Copy link
Contributor Author

taka61 commented Nov 29, 2021

@yoshitaku-jp さん

無事にユーザを見つけることができました!

ユーザー一覧に反映されていたようで良かったです〜🙏

また「ログイン」に関してですが、
ユーザ名:tatenoicon
パスワード:testtest
でログイン出来ましたので、こちらで試してみて下さい!

スクリーンショット 2021-11-29 11 43 15

スクリーンショット 2021-11-29 11 24 35

@yoshitaku-jp
Copy link
Contributor

@taka61 さん
私の方でもログイン確認できました!大丈夫です!

@taka61
Copy link
Contributor Author

taka61 commented Dec 2, 2021

@yoshitaku-jp さん
確認いただきありがとうございました!🙏

@komagata さん
チームメンバーのレビューを通過しましたので、お時間ある際にこちらのレビューをお願いします!

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.

conflictの修正をお願いします〜

@taka61
Copy link
Contributor Author

taka61 commented Dec 4, 2021

@komagata さん
コンフリクト解消しました!
ご確認をお願いします🙏

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 c7bf732 into main Dec 4, 2021
@komagata komagata deleted the feature/add_user_with_rectangular_icon_in_test_data branch December 4, 2021 18:55
@github-actions github-actions bot mentioned this pull request Dec 4, 2021
21 tasks
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.

None yet

3 participants