@hlfan commented on this pull request.
> @@ -11,14 +11,16 @@
<% end %>
<div class="d-flex align-items-end">
- <ul class="nav nav-tabs fs-6">
- <li class="nav-item">
- <%= link_to t("sessions.new.tab_title"), "#", :class => "nav-link
active" %>
- </li>
- <li class="nav-item">
- <%= link_to t("users.new.tab_title"), url_for(:action => :new,
:controller => :users, :referer => params[:referer]), :class => "nav-link" %>
- </li>
- </ul>
+ <% if @allow_signup %>
I'd write the condition here and in the partial as `unless allow_signup ==
false` to handle the nil case consistently.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7138#pullrequestreview-4445813560
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7138/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
