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

メンターメニューの中身を追加する #7767

Closed
komagata opened this issue May 8, 2024 · 7 comments
Closed

メンターメニューの中身を追加する #7767

komagata opened this issue May 8, 2024 · 7 comments

Comments

@komagata
Copy link
Member

komagata commented May 8, 2024

↓現状のメンターメニュー

image

↓の実際のメンターページにはもっとページがあるので、メンターメニューも同じようにリンクを作ってほしい。

スクリーンショット 2024-05-08 223736

@komagata
Copy link
Member Author

komagata commented May 8, 2024

📝 モバイル版のメニューデザインを調整するので完成したらまず @machida にメンションを送ること。

@Shrimprin
Copy link
Contributor

@komagata @machida
テストについて質問させてください。

今回実装するヘッダーリンクのメンターメニューのテストは書くべきでしょうか?
また、どこに書くべきでしょうか?
確認した限り、既に存在するヘッダーリンクの各リンク(ダッシュボードや登録車情報変更、管理者メニューなど)のテストは書かれていないようで、上記について迷っています。

どのように対応べきがご助言願えますでしょうか。

  • テストを書く or 書かない
  • 書く場合、どこに書くべきか?
    test/system/mentor配下?
  • 書く場合、メンターメニュー以外の各メニューについてもついでにテストを書くべきか?

今回実装する範囲(メンターメニュー)以外のテストまで書くと別イシューになりそうなため、ひとまずはメンターメニューのみのテストをtest/system/mentor配下に書くのが良いと考えています。
初のイシューのため勝手がわかっておらず、お手数おかけしますがご回答お願いいたします。

@komagata
Copy link
Member Author

@Shrimprin 書かなくてよいです〜。

@Shrimprin
Copy link
Contributor

@komagata
承知しました!
ご回答ありがとうございます ☺️

@Shrimprin
Copy link
Contributor

仕様についてDoscordにて確認した内容を記録としてこちらに記載します。

確認事項

ヘッダーリンクのメンターメニューにブログとブログ記事作成の両方のリンクを載せるか?

結論

ブログのリンクのみ残し、ブログ記事作成のリンクは消してよい

@komagata
Copy link
Member Author

komagata commented Jun 7, 2024

@Shrimprin 本番環境で確認しました~。

@Shrimprin
Copy link
Contributor

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

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants