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

user-iconを非React化した #8255

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

Conversation

Ryooo-k
Copy link
Contributor

@Ryooo-k Ryooo-k commented Dec 19, 2024

Issue

概要

UserIcon.jsxを非React化し、不要なファイルは削除しました。

削除ファイルと削除した理由

  1. user-icon.vuetalk.vueから使用されていましたが、talk.vueが未使用ファイルのため、user-icon.vueおよびtalk.vueを削除しました。
  2. UserIcon.jsxは非React化のため削除し、user-icon.jsに置き換えました。
  3. ui/UserGroup.jsxは独自にUserIconを定義してますが、UserGroup.jsxを使用していたFilterByArea.jsxは、このPRで非React化されており、UserGroup.jsxは未使用ファイルとなっていたため削除しました。

変更確認方法

  1. chore/user-icon-to-non-vue-react をローカルに取り込む
    i. git fetch origin pull/8255/head:chore/user-icon-to-non-vue-react
    ii. git checkout chore/user-icon-to-non-vue-react
  2. foreman start -f Procfile.dev でローカルサーバーを立ち上げる
  3. ユーザー名komagata、パスワードtesttestでログインする
  4. 特別イベントにアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-14 20 15 06
  1. 定期イベントにアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-14 20 15 24
  1. ブックマーク一覧にアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-19 12 58 48
  1. 提出物にアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-19 13 01 48
  1. 通知にアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-19 13 05 06
  1. ダッシュボードにアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-19 13 06 28
  1. 企業別ユーザー一覧にアクセスし、ユーザーアイコンが表示されるか確認する。
スクリーンショット 2024-12-19 13 14 28

Screenshot

変更前

特別イベント

スクリーンショット 2024-12-14 20 15 06

定期イベント

スクリーンショット 2024-12-14 20 15 24

ブックマーク一覧

スクリーンショット 2024-12-19 12 58 48

提出物

スクリーンショット 2024-12-19 13 01 48

通知

スクリーンショット 2024-12-19 13 05 06

ダッシュボード

スクリーンショット 2024-12-19 13 06 28

企業別ユーザー一覧

スクリーンショット 2024-12-19 13 14 28

変更後

このPRに伴う画面の変更はございません。

@Ryooo-k Ryooo-k self-assigned this Dec 19, 2024
@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Dec 19, 2024

@Judeeeee
お疲れ様です!
こちらのレビューをお願いしたいのですが可能でしょうか?
(現在、テストの不具合でregular_events_test.rbの2つが落ちる状態となっておりCIが通ってませんが、こちらで対応中となっております。)

@Ryooo-k Ryooo-k requested a review from Judeeeee December 19, 2024 04:35
@Judeeeee
Copy link
Contributor

@Ryooo-k
お疲れ様です!
依頼いただきありがとうございますー
折角の機会なのですが、私が年末まで対応できなさそうなので他の方に依頼いただけると助かります🙇
お手数ですが、よろしくお願いいたします🙏

@Ryooo-k
Copy link
Contributor Author

Ryooo-k commented Dec 20, 2024

@Judeeeee
承知しました!他の方に依頼いますね👍

@su-su-su-su
お疲れ様です。
こちらのレビューをお願いできますでしょうか。

@Ryooo-k Ryooo-k requested review from su-su-su-su and removed request for Judeeeee December 20, 2024 01:53
@Ryooo-k Ryooo-k marked this pull request as ready for review December 20, 2024 08:02
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.

2 participants