Skip to content

Commit

Permalink
ダッシュボード配下にWatch中のタブを追加
Browse files Browse the repository at this point in the history
  • Loading branch information
@rjtt17
rjtt17 committed Jul 23, 2021
1 parent aefef66 commit 9cdd169
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions app/views/home/_page_tabs.html.slim
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,7 @@
li.page-tabs__item
= link_to current_user_products_path, class: "page-tabs__item-link #{current_page_tab_or_not('products')}" do
| 自分の提出物
- if !user.staff? || user.watches.present?
li.page-tabs__item
= link_to current_user_watches_path, class: "page-tabs__item-link #{current_page_tab_or_not('watches')}" do
| 自分のWatch中

3 comments on commit 9cdd169

@masamune17
Copy link
Contributor

Choose a reason for hiding this comment

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

@rjtt17 さん、一点ご質問がありますので以下に記載いたします。

現在私も似たようなissueを実施しており、ダッシュボード配下にブックマークのタブを追加しているのですが、スタッフじゃない(!user.staff?)という条件は日報や提出物とは違い、必要ではないのかな？と考えております。

そこでブックマーク機能と似ている、Watch機能のタブを表示する条件に、スタッフじゃないという条件がついている理由についてお教え願います。

@rjtt17
Copy link
Contributor Author

@rjtt17 rjtt17 commented on 9cdd169 Sep 7, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

@masamune17
Watchしている物がない場合は、管理者、メンター、アドバイザーの方にタブを表示する必要が不要かなと思ったので、上記の実装にしています！　ただ、明確にこちらの表示について、駒形さんや町田さんに確認していないので、確認した方がいいかもしれません〜

@masamune17
Copy link
Contributor

Choose a reason for hiding this comment

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

@rjtt17 さん、ご返信ありがとうございます！！
理由について承知致しました。
駒形さんや町田さんに確認しながら実装いたします。

ご確認、ありがとうございました。

Please sign in to comment.