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

管理者が自分以外のユーザーアイコンを変更できないバグを修正 #8337

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

Conversation

kitarou888
Copy link

@kitarou888 kitarou888 commented Feb 13, 2025

Issue

概要

管理者でログインをすると、ユーザーの個別ページからそのユーザーのプロフィール編集ページに入ることができ、プロフィールの編集が行える。
しかし、現状管理者ではその機能を使ってユーザーのアイコンを変更しようとしても変更が反映されない。
そのバグの修正を行った。

変更確認方法

  1. bug/admin-cannot-update-user-iconをローカルに取り込む
    1. git fetch origin bug/admin-cannot-update-user-icon
    2. git switch bug/admin-cannot-update-user-icon
  2. foreman start -f Procfile.devでローカルサーバーを起動する
  3. ユーザー名machida、パスワードtesttestでログインする(管理者であれば誰でも可)
  4. hatsunoさんのプロフィールページにアクセスする
  5. 右下「管理者として情報変更」ボタンを押下する
  6. 「ユーザーアイコン」エリアの「画像を変更」ボタンを押下し、任意の画像をアップロードする
  7. 一番下「更新する」ボタンを押下する
  8. ユーザーアイコンがアップロードした画像に更新されていることを確認する

Screenshot

image image image image

@kitarou888 kitarou888 self-assigned this Feb 13, 2025
@kitarou888
Copy link
Author

@su-su-su-su
お疲れさまです!
お手隙の際にこちらのレビューをお願いできますでしょうか??
全然急ぎではなく、2週間以上かかっても問題ありませんので、ご検討のほどよろしくお願いします🙏

@su-su-su-su
Copy link
Contributor

@kitarou888
お疲れ様です!
レビューさせて頂きます。
期間は1週間程頂ければと思います!
よろしくお願い致します🙇

Copy link
Contributor

@su-su-su-su su-su-su-su left a comment

Choose a reason for hiding this comment

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

@kitarou888
お疲れ様です!
確認させていただきました!
問題なさそうなので、私からはapproveさせていただきます🙏

@kitarou888
Copy link
Author

@su-su-su-su
承知しました!
お忙しいなかレビューいただきありがとうございました🙏

@komagata
メンバーレビューが完了しましたので、レビューのほどよろしくお願いします!

@komagata
Copy link
Member

@okuramasafumi こちらのレビューをお願いできればありがたいです〜

Copy link
Contributor

@okuramasafumi okuramasafumi left a comment

Choose a reason for hiding this comment

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

良さそうです!
nativeメソッド、知らなかったのですが、
https://www.rubydoc.info/gems/capybara/Capybara/Node/Element#native-instance_method
のことなのでしょうか。他のテストコードでも利用例があるので問題はなさそうです。

@kitarou888
Copy link
Author

@okuramasafumi
レビューいただきありがとうございました🙏
nativeメソッドは他の方が書かれたテストコード(一般ユーザーによる画像アップロード)で使われていた処理の流用とはいえ、メソッド自体を調べられていませんでしたので、確認しておきます。

※参考にしたテストコード
test/system/users_test.rb'can upload heic image as user avatar'

@kitarou888
Copy link
Author

@okuramasafumi @komagata
mainにマージいただきましたらステージング環境を確認しますので、ご連絡願います!

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.

4 participants