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

メンターメニューの中身を追加 #7777

Merged
merged 1 commit into from
May 28, 2024

Conversation

Shrimprin
Copy link
Contributor

@Shrimprin Shrimprin commented May 15, 2024

Issue

概要

メンターページに存在するページの内、メンターメニューに無いリンクを追加しました。
具体的には以下のリンクを追加しました。

  • プラクティス
  • カテゴリー
  • コース
  • 紹介・言及記事
  • ブログ

また、下記はブログと内容が重複するため削除しました。

  • ブログ記事作成

変更確認方法

  1. chore/add-mentor-menu-contentsをローカルに取り込む
  2. bin/setupを実行
  3. foreman start -f Procfile.devでローカルサーバを立ち上げ
  4. ユーザー名mentormentaroでログイン
  5. 画面右上のユーザーアイコンをクリック
  6. 表示されるリストから下記を確認する

Screenshot

いずれもユーザーアイコンをクリックして表示されるリストです。
変更後はメンターメニューにリンクが追加されています。

変更前

image

変更後

image

@Shrimprin
Copy link
Contributor Author

@machida
プログラムを作成しましたのでデザインをお願いできますでしょうか 🙏

@Shrimprin
Copy link
Contributor Author

@mousu-a
もし可能でしたらレビューお願いできますでしょうか?
よろしくお願いいたします 🙏

@mousu-a
Copy link
Contributor

mousu-a commented May 22, 2024

@Shrimprin
お疲れ様です!
レビューに関してはぜひ引き受けさせていただきたいのですが、machidaさんのデザインが完了してからメンバーのレビュー、という流れの方がいいかもしれません!

お手数おかけして申し訳ないのですがその辺りmachidaさんかkomagataさんに確認していただければと思います🙇‍♂️

@Shrimprin
Copy link
Contributor Author

Shrimprin commented May 22, 2024

@mousu-a
説明不足で申し訳ございません!
開発MTGにてmachidaさんからは「デザインは不要のためメンバーのレビューに進んでOK」という説明を口頭で受けましたのでレビューを依頼させていただいています💦

急ぎではありませんので他に支障のないペースでレビューいただければと思います 🙏 よろしくお願いいたします!

@mousu-a
Copy link
Contributor

mousu-a commented May 22, 2024

@Shrimprin

開発MTGにてmachidaさんからは「デザインは不要のためメンバーのレビューに進んでOK」という説明を口頭で受けましたのでレビューを依頼させていただいています💦

お〜そうなんですね!すみません、こちらこそ確認不足でした🙏

ではレビュー進めさせていただきます〜😊
今日中には返せると思います!

@Shrimprin
Copy link
Contributor Author

@mousu-a
ありがとうございます!よろしくお願いします〜

@mousu-a
Copy link
Contributor

mousu-a commented May 22, 2024

@Shrimprin
おはようございます。
レビューお待たせいたしました!
変更確認方法わかりやすくとてもありがたいです〜😊

一点気になるところがありましたのでShrimprinさんのご意見を伺えればと思います🙏

「ブログ記事作成」は残した方がいいかも?

また、下記はブログと内容が重複するため削除しました。
  ブログ記事作成

とのことなのですが、自分としてはブログ記事作成はそのまま残しても良いのかなと考えています。

正直なところメンターの方々がどれだけ「ブログ記事作成」を使っているかは推測でしかわかりませんが、
「ブログ」は一覧へのアクセス、「ブログ記事作成」は新規作成画面へのアクセスとそれぞれニーズ(求めている役割)が違っていそうです。なので両方あっても良いのかなと思ったのですがどうでしょうか。
(確かにブログ→ブログの新規作成をすることはできますが、それだと初めからブログを新規作成したい時はワンアクション手間がかかるようになると思います)

Shrimprinさんのご意見を伺いたいです🙏

@Shrimprin
Copy link
Contributor Author

@mousu-a
早速のレビューありがとうございます!
返信が遅れて申し訳ございません 🙏

「ブログ」は一覧へのアクセス、「ブログ記事作成」は新規作成画面へのアクセスとそれぞれニーズ(求めている役割)が違っていそうです。なので両方あっても良いのかなと思ったのですがどうでしょうか。

こちらについてはkomagataさんに確認し、「ブログ記事作成」は削除してよいとのことだったのでこのような実装としています。
(この確認はDiscord上で行ってしまったのですが、komagataさんから「後から見る人が経緯が分かるように以降はGitHub上で確認すること」とアドバイス受けていました。まさにその通りで経緯がmousu-aさんと共有できていなかったためにお手を煩わせてしまいました😨すみません!)

@mousu-a
Copy link
Contributor

mousu-a commented May 23, 2024

@Shrimprin

(この確認はDiscord上で行ってしまったのですが、komagataさんから「後から見る人が経緯が分かるように以降はGitHub上で確認すること」とアドバイス受けていました。まさにその通りで経緯がmousu-aさんと共有できていなかったためにお手を煩わせてしまいました😨すみません!)

ドンピシャでしたね(笑)
今後の学びということで👍

それでしたらこちらからはApproveとさせていただきます〜!

ちなみにDiscordやMTGなどで行った確認や相談は、こちら(Github issue / PR)のコメント欄にログを残しておいてあげるとわかりやすくなるかもです〜🙆

ログ

MTGにてこのような確認 / 相談を行った。
このように実装してほしいとのこと。

みたいな感じでどうでしょうか👀

@Shrimprin
Copy link
Contributor Author

@mousu-a
レビューいただきありがとうございました!

ちなみにDiscordやMTGなどで行った確認や相談は、こちら(Github issue / PR)のコメント欄にログを残しておいてあげるとわかりやすくなるかもです〜🙆

GItHub issueの方にコメントで記載しました。ご助言いただきありがとうございます〜 🍵

今後のチーム開発でも引き続きよろしくお願いいたします 🙏

@Shrimprin Shrimprin requested a review from komagata May 23, 2024 14:42
@Shrimprin
Copy link
Contributor Author

@komagata
お疲れ様です。
メンバーからApproveいただきましたのでレビューお願いできますでしょうか。
よろしくお願いいたします 🙏

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 3719b7f into main May 28, 2024
7 checks passed
@komagata komagata deleted the chore/add-mentor-menu-contents branch May 28, 2024 19:56
@github-actions github-actions bot mentioned this pull request May 28, 2024
18 tasks
@Shrimprin
Copy link
Contributor Author

Shrimprin commented May 29, 2024

@komagata
ありがとうございます! 🙏

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