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

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

Closed
wants to merge 63 commits into from

Conversation

kitarou888
Copy link

Issue

概要

現状のbootcampアプリの検索機能は、「カテゴリ(日報、Docsなど)と検索ワード」のみを指定できる仕様だが、新たに「自分のもののみ」をフィルタリングできるチェックボックス機能を実装した。

変更確認方法

  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

変更前

変更後

Yuichiro Makita and others added 30 commits February 5, 2025 12:34
Co-authored-by: Masaki Komagata <[email protected]>
FAQ にデザインを入れた(ec58966)に対するconflictを解消
研修生向けの機能。設定されていても課題の提出・閲覧は可能
fixtureデータのconflictを修正
commit 69af563 とconflictしていたので修正
ユーザーの無料フラグを削除するようにした

プラクティスメモのconflictを解消
form_tabs.sass のconflictを解消
reports/show.html のconflictを解消
app/views/users/_form.html.slim のconflictを解消
lintの結果にもとづいた修正も含む
ユーザー情報と研修生からみたプラクティス一覧/詳細ページのテストを追加
スキップしたプラクティスは進捗率の分母に含まないように変更
a6b0c80 とのconflict解消
カスタムvalidateのpractice_belongs_to_userはsystemでは起こり得ない状態なので削除
kyokucho1989 and others added 21 commits February 12, 2025 11:16
categoryies_with_uniq_practicesをUserPracticeクラスへ移行
方針用にあげているcommitはあとで整理します
メソッド名をわかりやすくするためsorted_practicesに変更
withがついていて、二つのことを表現してしまっているため変更
userモデルのpracticeメソッドを参照していたため、記述を変更
ユーザー一覧の修正 (e1ffae5) で発生したconflictを解消
ユーザーに関するプラクティスのメソッドを移行していたのでそれに合わせて変更
issue No. 7994 用のテストにあうようにファイルを編集
@kitarou888 kitarou888 self-assigned this Feb 12, 2025
@kitarou888 kitarou888 closed this Feb 12, 2025
@kitarou888 kitarou888 deleted the bug/admin-cannot-update-user-icon branch February 12, 2025 10:29
@kitarou888 kitarou888 restored the bug/admin-cannot-update-user-icon branch February 12, 2025 10:30
@kitarou888 kitarou888 deleted the bug/admin-cannot-update-user-icon branch February 12, 2025 10:33
@kitarou888 kitarou888 restored the bug/admin-cannot-update-user-icon branch February 12, 2025 10:35
@komagata
Copy link
Member

@kitarou888 どういう流れ(経緯)でCloseするに至ったのかをここに残しておいていただければありがたいです。
(後から参加したメンバーが遡ってみたときにわかるように)

@kitarou888
Copy link
Author

kitarou888 commented Feb 13, 2025

@komagata
本PRがClosedするに至った経緯を説明します。

  • 作業ブランチbug/admin-cannot-update-user-icon をpushし、PRを作成したところ、CIのtestが落ちていた。
  • CIを再試行する方法がわからなかったため、再度pushするためにすでにpushした上記ブランチを削除した。
  • 上記ブランチを削除したことで、同ブランチに紐づいた本PRがclosedになってしまった。
  • 本PRをReopenしようとするも方法がわからず。(下段にある「Reopen pull request」ボタンを試すもうまくいかず)

@kitarou888 kitarou888 deleted the bug/admin-cannot-update-user-icon branch February 13, 2025 11:51
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.

6 participants