-
Notifications
You must be signed in to change notification settings - Fork 7
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
Flexibus/#374/user dashboard #567
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
config/locales/de.yml
Outdated
@@ -111,7 +111,8 @@ de: | |||
edit_profile: "Profil bearbeiten" | |||
|
|||
dashboard: | |||
title: "Dashboard für %{name}" | |||
title: "Hallo %{name}!" | |||
show_all: "Alle anzeigem" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
@@ -2,3 +2,14 @@ | |||
<div class="page-header"> | |||
<h1><%= t('dashboard.title', name: @user.first_name) %></h1> | |||
</div> | |||
|
|||
<% if user_signed_in? and current_user.teams_created_by_user.present? %> | |||
<h4><%= t('teams.index.my_teams')%> <a><%= t('dashboard.show_all')%></a></h4> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
href
is missing -> link doesn't work
Number of teams shown is not limited to 4 - is that inteded or ok for now? @emanuel-metzenthin @zhamanakov |
Number of teams is ok, as no other sections are implemented yet. |
Closes #374.
The dashboard is not finished. But it already contains the user's teams. This can be merged before the feature freeze and final presentation.